diff options
author | Peter Wu <peter@lekensteyn.nl> | 2018-05-09 22:37:41 +0200 |
---|---|---|
committer | Peter Wu <peter@lekensteyn.nl> | 2018-05-14 13:44:23 +0000 |
commit | 10306f9a32ee4ca65283110ac286bb1ba9ef2a01 (patch) | |
tree | 05e4b1ac1b9387f00007ad583571d2a57566c0e7 /epan/dfilter | |
parent | bb81e1807b2a5b67905cfa1594dbbb535fe8b37e (diff) |
Free g_array_free-related memory leaks
g_array_free(a, FALSE) returns "a->data". Callers that do not handle
this will leak memory. Convert other users to use the return value
instead of direct access to "a->data".
Change-Id: I0a29864e8106c0bf09e9573ef29e4474179c4171
Reviewed-on: https://code.wireshark.org/review/27438
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Diffstat (limited to 'epan/dfilter')
-rw-r--r-- | epan/dfilter/dfilter-macro.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/epan/dfilter/dfilter-macro.c b/epan/dfilter/dfilter-macro.c index f94970a895..5ff2493489 100644 --- a/epan/dfilter/dfilter-macro.c +++ b/epan/dfilter/dfilter-macro.c @@ -412,9 +412,7 @@ done: m->parts = (gchar **)g_ptr_array_free(parts, FALSE); g_free(m->args_pos); - m->args_pos = (int*)(void *)args_pos->data; - - g_array_free(args_pos,FALSE); + m->args_pos = (int*)(void *)g_array_free(args_pos, FALSE); m->argc = argc; |