Fixed #57 crash for get alarm summary. Thank you Walter! (#76)

Co-authored-by: Steve Karg <skarg@users.sourceforge.net>
2020-04-23 07:29:56 -05:00
parent 5e9cb90a09
commit 62bfeba2c5
1 changed files with 6 additions and 1 deletions
@@ -381,7 +381,12 @@ uint16_t apdu_decode_confirmed_service_request(uint8_t *apdu, /* APDU data */
                return 0;
            }
        }
-        if (apdu_len >= (len+2)) {
+        if (apdu_len == (len + 1)) {
+            /* no request data as seen with Inneasoft BACnet Explorer */
+            *service_choice = apdu[len++];
+            *service_request = NULL;
+            *service_request_len = 0;
+        } else if (apdu_len >= (len+2)) {
            *service_choice = apdu[len++];
            *service_request = &apdu[len];
            *service_request_len = apdu_len - len;