Commit 29693b83 authored by Nick Mathewson's avatar Nick Mathewson 🤹
Browse files

Make sure that we free 'addr' at the end of a pair of addr tests 

Fixes a couple of Coverity warnings about possible memory leaks.
Bug not in any released Tor.
parent cca9e1c8

src/test/test_addr.c

+2 −4
Original line number Diff line number Diff line
@@ -1751,10 +1751,9 @@ test_addr_is_valid(void *arg) 
  get_af_unspec(test_addr);

  TEST_ADDR_VALIDITY(test_addr, 0, 0);

  TEST_ADDR_VALIDITY(test_addr, 1, 0);

  tor_free(test_addr);



 done:

  ;

  tor_free(test_addr);

}



#define TEST_ADDR_IS_NULL(a, rv) STMT_BEGIN                 \
@@ -1797,10 +1796,9 @@ test_addr_is_null(void *arg) 
  /* Test for address family AF_UNSPEC. */

  get_af_unspec(test_addr);

  TEST_ADDR_IS_NULL(test_addr, 1);

  tor_free(test_addr);



 done:

  ;

  tor_free(test_addr);

}



#ifndef COCCI