Only mark handle as completed when fully received.

3 files changed, 8 insertions, 1 deletions

diff --git a/src/isotp/isotp.c b/src/isotp/isotp.c
index 9aa051bc..1605d435 100644
--- a/src/isotp/isotp.c
+++ b/src/isotp/isotp.c
@@ -76,6 +76,8 @@ IsoTpMessage isotp_receive_can_frame(IsoTpShims* shims, IsoTpHandle* handle,
             message.payload = payload;
             message.size = payload_length;
             message.completed = true;
+            handle->success = true;
+            handle->completed = true;
             isotp_handle_single_frame(handle, &message);
             break;
          }
diff --git a/src/isotp/isotp.h b/src/isotp/isotp.h
index 103c402d..c0db0178 100644
--- a/src/isotp/isotp.h
+++ b/src/isotp/isotp.h
@@ -101,6 +101,11 @@ IsoTpMessage isotp_receive_can_frame(IsoTpShims* shims, IsoTpHandle* handle,
         const uint16_t arbitration_id, const uint8_t data[],
         const uint8_t size);
 
+// TODO perhaps this makes more sense as 2 functions:
+// bool isotp_continue_send()
+// IsoTpMessage isotp_continue_receive()
+// but both with the same args
+
 /* Public: Change the timeout for waiting on an ISO-TP response frame.
  *
  * If this function is not used, the conventional 100ms is used by default.
diff --git a/src/isotp/receive.c b/src/isotp/receive.c
index 1692f8d3..b07950c5 100644
--- a/src/isotp/receive.c
+++ b/src/isotp/receive.c
@@ -20,7 +20,7 @@ IsoTpHandle isotp_receive(IsoTpShims* shims,
 
     IsoTpHandle handle = {
         success: false,
-        completed: true,
+        completed: false,
         receive_handle: receive_handle,
         type: ISOTP_HANDLE_RECEIVING
     };