[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] Coverity: fix memory leak on error in test function.



commit ea154a6108bae597cb37e6bc53036b6dd2ed6187
Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date:   Fri Jun 28 12:27:51 2019 -0400

    Coverity: fix memory leak on error in test function.
    
    The function make_intro_from_plaintext() in test_introduce.c would
    leak memory if we ever hit a failure from our underlying crypto
    functions.  This kind of failure should be impossible, but it's best
    to be safe here.
    
    Bugfix on 0.2.4.1-alpha.
---
 src/test/test_introduce.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/test/test_introduce.c b/src/test/test_introduce.c
index 4a6d90d97..104e973b1 100644
--- a/src/test/test_introduce.c
+++ b/src/test/test_introduce.c
@@ -383,8 +383,10 @@ make_intro_from_plaintext(
 
   /* Output the cell */
   *cell_out = cell;
+  cell = NULL;
 
  done:
+  tor_free(cell);
   return cell_len;
 }
 
@@ -535,4 +537,3 @@ struct testcase_t introduce_tests[] = {
   INTRODUCE_LEGACY(late_parse_v3),
   END_OF_TESTCASES
 };
-



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits