diff options
| author | Bill Meier <wmeier@newsguy.com> | 2015-01-25 15:51:01 -0500 |
|---|---|---|
| committer | Bill Meier <wmeier@newsguy.com> | 2015-01-25 22:04:42 +0000 |
| commit | be798e18f4bab165944f6d5ab0ff661f09ea7870 (patch) | |
| tree | 8b589220492c8367f317b8982d5f669457674e27 /epan/dissectors/packet-pana.c | |
| parent | 065304cae189c7e00e9dafd650575efaaf48b2fb (diff) | |
pana, gsm_abis_oml, rsl, netrom: Remove 'if (tree)'
Thou shalt call col_...() and/or a dissector under 'if(tree)'
Change-Id: Ibe62bc8229cb759d5f8b092aa7143d8e2f799037
Reviewed-on: https://code.wireshark.org/review/6781
Reviewed-by: Bill Meier <wmeier@newsguy.com>
Diffstat (limited to 'epan/dissectors/packet-pana.c')
| -rw-r--r-- | epan/dissectors/packet-pana.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/epan/dissectors/packet-pana.c b/epan/dissectors/packet-pana.c index 8381530f49..a326e68072 100644 --- a/epan/dissectors/packet-pana.c +++ b/epan/dissectors/packet-pana.c @@ -630,9 +630,7 @@ dissect_pana_pdu(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) avp_tvb = tvb_new_subset_length(tvb, offset, avp_length); avp_tree = proto_tree_add_subtree(pana_tree, tvb, offset, avp_length, ett_pana_avp, NULL, "Attribute Value Pairs"); - if (avp_tree != NULL) { - dissect_avps(avp_tvb, pinfo, avp_tree); - } + dissect_avps(avp_tvb, pinfo, avp_tree); } } |