Merge pull request #7678 from bandi13/fixReturnType

ret will be set to 1 (WOLFSSL_SUCCESS), the rest checks for 'ret == 0'
wolfSSL · Jun 25, 2024 · 38335f4 · 38335f4
2 parents 263eb6c + 773451a
commit 38335f4
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/internal.c b/src/internal.c
@@ -36661,8 +36661,12 @@ static int DoSessionTicket(WOLFSSL* ssl, const byte* input, word32* inOutIdx,
 
             /* check for TLS_EMPTY_RENEGOTIATION_INFO_SCSV suite */
             ret = TLSX_AddEmptyRenegotiationInfo(&ssl->extensions, ssl->heap);
-            if (ret != WOLFSSL_SUCCESS)
+            if (ret != WOLFSSL_SUCCESS) {
+                ret = SECURE_RENEGOTIATION_E;
                 goto out;
+            } else {
+                ret = 0;
+            }
 
             extension = TLSX_Find(ssl->extensions, TLSX_RENEGOTIATION_INFO);
             if (extension) {