diff options
| author | Gerald Combs <gerald@wireshark.org> | 2015-08-06 11:23:55 -0700 |
|---|---|---|
| committer | Gerald Combs <gerald@wireshark.org> | 2015-08-06 21:22:13 +0000 |
| commit | ff24ecceb976e25cf58040b116f25fc218290540 (patch) | |
| tree | b6568500814f3dde7bd88e3433af58c503ba9693 /asn1/camel | |
| parent | 9fd3bcc25e0b6c1feaf6e20850200de4acf5637a (diff) | |
| download | wireshark-ff24ecceb976e25cf58040b116f25fc218290540.tar.gz | |
Fixup stat_tap table freeing.
As it turns out we *do* need to free table elements. We also need to
free the tables themselves and clear the table array. Do so.
Change-Id: Ic1c81388eac8f47f74caea0169c79685a83aaff9
Reviewed-on: https://code.wireshark.org/review/9901
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'asn1/camel')
| -rw-r--r-- | asn1/camel/packet-camel-template.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/asn1/camel/packet-camel-template.c b/asn1/camel/packet-camel-template.c index 152aabad7c..1d77cb779a 100644 --- a/asn1/camel/packet-camel-template.c +++ b/asn1/camel/packet-camel-template.c @@ -1285,6 +1285,12 @@ camel_stat_reset(new_stat_tap_table* table) } } +static void +camel_stat_free_table_item(new_stat_tap_table* table _U_, guint row _U_, guint column, stat_tap_table_item_type* field_data) +{ + if (column != MESSAGE_TYPE_COLUMN) return; + g_free((char*)field_data->value.string_value); +} /*--- proto_reg_handoff_camel ---------------------------------------*/ static void range_delete_callback(guint32 ssn) @@ -1537,7 +1543,7 @@ void proto_register_camel(void) { camel_stat_init, camel_stat_packet, camel_stat_reset, - NULL, + camel_stat_free_table_item, NULL, sizeof(camel_stat_fields)/sizeof(stat_tap_table_item), camel_stat_fields, sizeof(camel_stat_params)/sizeof(tap_param), camel_stat_params, |