aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dfilter
diff options
context:
space:
mode:
authorPeter Wu <peter@lekensteyn.nl>2018-05-09 22:37:41 +0200
committerPeter Wu <peter@lekensteyn.nl>2018-05-14 13:44:23 +0000
commit10306f9a32ee4ca65283110ac286bb1ba9ef2a01 (patch)
tree05e4b1ac1b9387f00007ad583571d2a57566c0e7 /epan/dfilter
parentbb81e1807b2a5b67905cfa1594dbbb535fe8b37e (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.c4
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;