Fix leaking EVP_MAC_CTX in one shot (#107471)

Co-authored-by: Kevin Jones <kevin@vcsjones.com>
dotnet · Sep 9, 2024 · c608194 · c608194
1 parent 7b91da6
commit c608194
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/src/native/libs/System.Security.Cryptography.Native/pal_evp_mac.c b/src/native/libs/System.Security.Cryptography.Native/pal_evp_mac.c
@@ -369,32 +369,35 @@ int32_t CryptoNative_EvpMacOneShot(EVP_MAC* mac,
 
         params[i] = OSSL_PARAM_construct_end();
 
+        int32_t ret = 0;
+
         if (!EVP_MAC_init(ctx, NULL, 0, params))
         {
-            EVP_MAC_CTX_free(ctx);
-            return 0;
+            goto done;
         }
 
         if (!EVP_MAC_update(ctx, data, dataLengthT))
         {
-            EVP_MAC_CTX_free(ctx);
-            return 0;
+            goto done;
         }
 
         size_t written = 0;
 
         if (!EVP_MAC_final(ctx, destination, &written, macLengthT))
         {
-            EVP_MAC_CTX_free(ctx);
-            return 0;
+            goto done;
         }
 
         if (written != macLengthT)
         {
-            return -3;
+            ret = -3;
+            goto done;
         }
 
-        return 1;
+        ret = 1;
+done:
+        EVP_MAC_CTX_free(ctx);
+        return ret;
     }
 #else
     (void)mac;