diff options
| author | Nick Mathewson <nickm@torproject.org> | 2013-03-21 07:52:36 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2013-03-21 07:52:36 -0400 |
| commit | 42fb61d172b172687cd57c86dd72b117f03f2136 (patch) | |
| tree | 47cd55050dd6af2d3fc140f079310fb24270a1e8 /src/test/test_util.c | |
| parent | 6196d0e83d78e2e8efff575d490f4cb254415832 (diff) | |
| download | tor-42fb61d172b172687cd57c86dd72b117f03f2136.tar tor-42fb61d172b172687cd57c86dd72b117f03f2136.tar.gz | |
Fix a small memory leak in the unit tests
Found by coverity; this is CID 992692.
Diffstat (limited to 'src/test/test_util.c')
| -rw-r--r-- | src/test/test_util.c | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/src/test/test_util.c b/src/test/test_util.c index 7ab54e153..2c4f8040d 100644 --- a/src/test/test_util.c +++ b/src/test/test_util.c @@ -802,7 +802,7 @@ test_util_strmisc(void) { char buf[1024]; int i; - char *cp; + char *cp, *cp_tmp; /* Test strl operations */ test_eq(5, strlcpy(buf, "Hello", 0)); @@ -1005,20 +1005,20 @@ test_util_strmisc(void) /* Test strndup and memdup */ { const char *s = "abcdefghijklmnopqrstuvwxyz"; - cp = tor_strndup(s, 30); - test_streq(cp, s); /* same string, */ - test_neq_ptr(cp, s); /* but different pointers. */ - tor_free(cp); + cp_tmp = tor_strndup(s, 30); + test_streq(cp_tmp, s); /* same string, */ + test_neq_ptr(cp_tmp, s); /* but different pointers. */ + tor_free(cp_tmp); - cp = tor_strndup(s, 5); - test_streq(cp, "abcde"); - tor_free(cp); + cp_tmp = tor_strndup(s, 5); + test_streq(cp_tmp, "abcde"); + tor_free(cp_tmp); s = "a\0b\0c\0d\0e\0"; - cp = tor_memdup(s,10); - test_memeq(cp, s, 10); /* same ram, */ - test_neq_ptr(cp, s); /* but different pointers. */ - tor_free(cp); + cp_tmp = tor_memdup(s,10); + test_memeq(cp_tmp, s, 10); /* same ram, */ + test_neq_ptr(cp_tmp, s); /* but different pointers. */ + tor_free(cp_tmp); } /* Test str-foo functions */ @@ -1097,7 +1097,7 @@ test_util_strmisc(void) tt_int_op(strcmp_len("blah", "", 0), ==, 0); done: - ; + tor_free(cp_tmp); } static void |