diff options
| author | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2005-07-24 02:43:41 +0000 |
|---|---|---|
| committer | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2005-07-24 02:43:41 +0000 |
| commit | 835df3a1d2aef34df8468ce3046ad1c90333d7a0 (patch) | |
| tree | d22834fc143b9ed887fac9996a48a5849e57454a /epan/dissectors/packet-fix.c | |
| parent | 910f98eb86c4662c25a66106f502395f18304a83 (diff) | |
convert some uses of tvb_get_string() to ep_tvb_get_string()
there was at least one obvious memleak in one of the uses of tvb_get_string()
svn path=/trunk/; revision=15027
Diffstat (limited to 'epan/dissectors/packet-fix.c')
| -rw-r--r-- | epan/dissectors/packet-fix.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/epan/dissectors/packet-fix.c b/epan/dissectors/packet-fix.c index aeacf739ca..beae7737b9 100644 --- a/epan/dissectors/packet-fix.c +++ b/epan/dissectors/packet-fix.c @@ -868,9 +868,8 @@ dissect_fix(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) } if (check_col(pinfo->cinfo, COL_INFO)) { - value = tvb_get_string(tvb, value_offset, value_len); + value = ep_tvb_get_string(tvb, value_offset, value_len); col_add_fstr(pinfo->cinfo, COL_INFO, "%s", (char *)g_datalist_get_data(&msg_types, value)); - g_free(value); } /* In the interest of speed, if "tree" is NULL, don't do any work not @@ -926,11 +925,10 @@ dissect_fix(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) packet. */ return TRUE; } - tag_str = tvb_get_string(tvb, field_offset, tag_len); + tag_str = ep_tvb_get_string(tvb, field_offset, tag_len); tag = atoi(tag_str); - g_free(tag_str); - value = tvb_get_string(tvb, value_offset, value_len); + value = ep_tvb_get_string(tvb, value_offset, value_len); switch(tag) { case 1: /* Field Account */ @@ -2916,7 +2914,6 @@ dissect_fix(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) field_offset = offset = ctrla_offset + 1; ctrla_offset = tvb_find_guint8(tvb, field_offset, -1, 0x01); - g_free(value); tag_str = NULL; } } |