Parcourir la source

Improve sequence number skipping for fuzzer

Stefan Profanter il y a 7 ans
Parent
commit
7744bdfd7b
1 fichiers modifiés avec 5 ajouts et 2 suppressions
  1. 5 2
      src/ua_securechannel.c

+ 5 - 2
src/ua_securechannel.c

@@ -976,14 +976,17 @@ UA_SecureChannel_processChunk(UA_SecureChannel *channel, UA_ByteString *chunk,
     if(retval != UA_STATUSCODE_GOOD)
         return retval;
 
-    /* Skip sequence number checking for fuzzer to improve coverage */
-#if !defined(FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION)
     /* Check the sequence number */
     if(sequenceNumberCallback == NULL)
         return UA_STATUSCODE_BADINTERNALERROR;
     retval = sequenceNumberCallback(channel, sequenceNumber);
+
+    /* Skip sequence number checking for fuzzer to improve coverage */
     if(retval != UA_STATUSCODE_GOOD)
+#if !defined(FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION)
         return retval;
+#else
+        retval = UA_STATUSCODE_GOOD;
 #endif
 
     /* Process the payload */