4a600f877
Guilherme Maciel Ferreira
2013-03-15
Subject
traceroute: free some memory allocated by xzalloc()
Binary
No binary (build failed)
Architecture & Compiler Flags
—
Build Log
(log yok)
Commit Diff GitHub →
networking/traceroute.c+13 -2
| @@ -805,6 +805,7 @@ common_traceroute_main(int op, char **argv) | |
| char *waittime_str; | |
| char *pausemsecs_str; | |
| char *first_ttl_str; | |
| + | char *dest_str; |
| #if ENABLE_FEATURE_TRACEROUTE_SOURCE_ROUTE | |
| llist_t *source_route_list = NULL; | |
| int lsrr = 0; | |
| @@ -1059,8 +1060,12 @@ common_traceroute_main(int op, char **argv) | |
| xsetgid(getgid()); | |
| xsetuid(getuid()); | |
| - | printf("traceroute to %s (%s)", argv[0], |
| - | xmalloc_sockaddr2dotted_noport(&dest_lsa->u.sa)); |
| + | dest_str = xmalloc_sockaddr2dotted_noport(&dest_lsa->u.sa); |
| + | printf("traceroute to %s (%s)", argv[0], dest_str); |
| + | if (ENABLE_FEATURE_CLEAN_UP) { |
| + | free(dest_str); |
| + | } |
| + | |
| if (op & OPT_SOURCE) | |
| printf(" from %s", source); | |
| printf(", %d hops max, %d byte packets\n", max_ttl, packlen); | |
| @@ -1216,6 +1221,12 @@ common_traceroute_main(int op, char **argv) | |
| } | |
| } | |
| + | if (ENABLE_FEATURE_CLEAN_UP) { |
| + | free(to); |
| + | free(lastaddr); |
| + | free(from_lsa); |
| + | } |
| + | |
| return 0; | |
| } | |