diff options
author | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2016-06-12 19:00:37 -0400 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2016-06-13 13:55:43 +0000 |
commit | c016dcdbf9b8037d2e2a8b544c5ebe92485e7ea5 (patch) | |
tree | 51c5ff38b555a0cd3630918da3551ebef9b13757 /epan/dissectors/packet-diameter.c | |
parent | 8b6ea372c1ba66667a985d3f90d3daef131bb1ed (diff) |
Diameter: free the diameter directory variable.
It's only needed for a few milliseconds--there's no need for it to stick around
until the epan scope ends.
As discussed on I51813815babb4c40722c38459139ab9e3e3ccb42.
Change-Id: I81fef351ef2d700cc5ec5866340605704173fbdb
Reviewed-on: https://code.wireshark.org/review/15861
Petri-Dish: Jeff Morriss <jeff.morriss.ws@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-diameter.c')
-rw-r--r-- | epan/dissectors/packet-diameter.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/dissectors/packet-diameter.c b/epan/dissectors/packet-diameter.c index 2db13ef06b..60a596aa53 100644 --- a/epan/dissectors/packet-diameter.c +++ b/epan/dissectors/packet-diameter.c @@ -1902,9 +1902,10 @@ dictionary_load(void) } /* load the dictionary */ - dir = wmem_strdup_printf(wmem_epan_scope(), "%s" G_DIR_SEPARATOR_S "diameter" G_DIR_SEPARATOR_S, get_datafile_dir()); + dir = wmem_strdup_printf(NULL, "%s" G_DIR_SEPARATOR_S "diameter" G_DIR_SEPARATOR_S, get_datafile_dir()); /* XXX We don't call ddict_free anywhere. */ d = ddict_scan(dir,"dictionary.xml",do_debug_parser); + wmem_free(NULL, dir); if (d == NULL) { g_hash_table_destroy(vendors); g_array_free(vnd_shrt_arr, TRUE); |