diff options
author | Michael Mann <mmann78@netscape.net> | 2017-02-24 22:11:22 -0500 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-02-25 13:35:26 +0000 |
commit | 924e6d2b49adf8eee1b13dba0feedadcc531245a (patch) | |
tree | 4cdc6aa1e05d10efffdc43c237b0c72b56c76504 /epan | |
parent | 762bf10834da9f8a5d51f2ff035970942d9070bf (diff) |
packet-fw1.c: header should be packet scoped, not epan scoped
Subtle memory leak
Change-Id: I39e3f2d35de5a8c48469f3e0992cdfec09ff0755
Reviewed-on: https://code.wireshark.org/review/20269
Petri-Dish: Michael Mann <mmann78@netscape.net>
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-fw1.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/epan/dissectors/packet-fw1.c b/epan/dissectors/packet-fw1.c index 208821a8cb..ac9e94b034 100644 --- a/epan/dissectors/packet-fw1.c +++ b/epan/dissectors/packet-fw1.c @@ -137,16 +137,13 @@ dissect_fw1(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* data _U_) static const char fw1_header[] = "FW1 Monitor"; ethertype_data_t ethertype_data; - header = wmem_strbuf_new_label(wmem_epan_scope()); + header = wmem_strbuf_new_label(wmem_packet_scope()); wmem_strbuf_append(header, fw1_header); /* Make entries in Protocol column and Info column on summary display */ col_set_str(pinfo->cinfo, COL_PROTOCOL, "FW1"); col_clear(pinfo->cinfo, COL_INFO); - - /* g_snprintf(header, sizeof(header), fw1_header); */ - /* fetch info to local variable */ direction = tvb_get_guint8(tvb, 0); |