ensure payload_len is zeroed on start of a new multi-frame packet

canard.c

@@ -544,6 +544,7 @@ int16_t canardHandleRxFrame(CanardInstance* ins, const CanardCANFrame* frame, ui
 
         // take off the crc and store the payload
         rx_state->timestamp_usec = timestamp_usec;
+        rx_state->payload_len = 0;
         const int16_t ret = bufferBlockPushBytes(&ins->allocator, rx_state, frame->data + 2,
                                                  (uint8_t) (frame->data_len - 3));
         if (ret < 0)

canard/service_client.h

@@ -68,7 +68,10 @@ class Client : public HandlerList, public Sender {
     /// @param transfer transfer object of the request
     void handle_message(const CanardRxTransfer& transfer) override {
         rsptype msg {};
-        rsptype::cxx_iface::rsp_decode(&transfer, &msg);
+        if (rsptype::cxx_iface::rsp_decode(&transfer, &msg)) {
+            // invalid decode
+            return;
+        }
 
         // scan through the list of entries for corresponding server node id and transfer id
         Client<rsptype>* entry = branch_head[index];

canard/service_server.h

@@ -51,7 +51,10 @@ class Server : public HandlerList {
     /// @param transfer transfer object of the request
     void handle_message(const CanardRxTransfer& transfer) override {
         reqtype msg {};
-        reqtype::cxx_iface::req_decode(&transfer, &msg);
+        if (reqtype::cxx_iface::req_decode(&transfer, &msg)) {
+            // invalid decode
+            return;
+        }
         transfer_id = transfer.transfer_id;
         // call the registered callback
         cb(transfer, msg);

canard/subscriber.h

@@ -71,7 +71,10 @@ class Subscriber : public HandlerList {
     /// @param transfer transfer object
     void handle_message(const CanardRxTransfer& transfer) override {
         msgtype msg {};
-        msgtype::cxx_iface::decode(&transfer, &msg);
+        if (msgtype::cxx_iface::decode(&transfer, &msg)) {
+            // invalid decode
+            return;
+        }
         // call all registered callbacks in one go
         Subscriber<msgtype>* entry = branch_head[index];
         while (entry != nullptr) {