fix mem leak on deallocating recv sessions

cisco · Aug 29, 2021 · 90d2345 · 90d2345
1 parent f8a374b
commit 90d2345
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/test/srtp_driver.c b/test/srtp_driver.c
@@ -2107,14 +2107,14 @@ srtp_err_status_t srtp_validate_cryptex()
         if (srtp_octet_string_is_eq(packet, reference, len)) {
             return srtp_err_status_fail;
         }
-    }
 
-    status = srtp_dealloc(srtp_snd);
-    if (status) {
-        return status;
+        status = srtp_dealloc(srtp_recv);
+        if (status) {
+            return status;
+        }
     }
 
-    status = srtp_dealloc(srtp_recv);
+    status = srtp_dealloc(srtp_snd);
     if (status) {
         return status;
     }
@@ -2584,14 +2584,14 @@ srtp_err_status_t srtp_validate_gcm_cryptex()
         if (srtp_octet_string_is_eq(packet, reference, len)) {
             return srtp_err_status_fail;
         }
-    }
 
-    status = srtp_dealloc(srtp_snd);
-    if (status) {
-        return status;
+        status = srtp_dealloc(srtp_recv);
+        if (status) {
+            return status;
+        }
     }
 
-    status = srtp_dealloc(srtp_recv);
+    status = srtp_dealloc(srtp_snd);
     if (status) {
         return status;
     }