diff options
author | Michael Mann <mmann78@netscape.net> | 2016-10-10 23:36:25 -0400 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2016-10-12 00:56:15 +0000 |
commit | 31a35aa9e6d48ea1ee179b26c1b11d7e6e8079a1 (patch) | |
tree | aaecd790d04e6f6c32e9a01e9b17e687acd16f80 /epan/packet.c | |
parent | a1ecd5661f4c28ca38bf83519e89bfbb2f502671 (diff) |
Cleanup from adding Decode As TCP preferences.
1. Fix bug caught by scan build (module not read)
2. Remove range preference call that should have been part of original patch.
Change-Id: I24b1fb253548bffddc4c8ebfc3ede666d8fd9dcd
Reviewed-on: https://code.wireshark.org/review/18143
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/packet.c')
-rw-r--r-- | epan/packet.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/packet.c b/epan/packet.c index 4a248821bf..50b24d7d85 100644 --- a/epan/packet.c +++ b/epan/packet.c @@ -1063,7 +1063,7 @@ void dissector_add_uint_range_with_preference(const char *abbrev, const char* ra /* Some preference callback functions use the proto_reg_handoff_ routine to apply preferences, which could duplicate the registration of a preference. Check for that here */ - else if (prefs_find_preference(module, abbrev) == NULL) { + if (prefs_find_preference(module, abbrev) == NULL) { description = wmem_strdup_printf(wmem_epan_scope(), "%s %s(s)", proto_get_protocol_short_name(handle->protocol), pref_dissector_table->ui_name); title = wmem_strdup_printf(wmem_epan_scope(), "%s(s)", pref_dissector_table->ui_name); |