ssl: don't say we consumed bytes if we didn't consume them

victorjulien · Apr 27, 2020 · 2dd0614 · 2dd0614
1 parent 38fe42e
commit 2dd0614
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/app-layer-ssl.c b/src/app-layer-ssl.c
@@ -505,6 +505,7 @@ static inline int TlsDecodeHSCertificateAddCertToChain(SSLState *ssl_state,
     return 0;
 }
 
+/** \retval consumed bytes consumed or -1 on error */
 static int TlsDecodeHSCertificate(SSLState *ssl_state,
                                   const uint8_t * const initial_input,
                                   const uint32_t input_len)
@@ -514,7 +515,7 @@ static int TlsDecodeHSCertificate(SSLState *ssl_state,
     Asn1Generic *cert = NULL;
 
     if (!(HAS_SPACE(3)))
-        return 1;
+        return 0;
 
     uint32_t cert_chain_len = *input << 16 | *(input + 1) << 8 | *(input + 2);
     input += 3;