Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 36614: /trunk/ /trunk/epan/dissectors/: packet-snmp.c /t

Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 36614: /trunk/ /trunk/epan/dissector

From: Stig Bjørlykke <stig@xxxxxxxxxxxxx>

Date: Wed, 13 Apr 2011 07:57:35 +0200

On Wed, Apr 13, 2011 at 12:12 AM,  <sfisher@xxxxxxxxxxxxx> wrote:
> Log:
>  Fix Coverity 1047: UNUSED_VALUE in dissect_smux

I don't think this is the correct fix, because now the
proto_item_add_subtree() has no function.
I think maybe smux_tree should be used in dissect_SMUX_PDUs_PDU().

What do you think?

-- 
Stig Bjørlykke

Follow-Ups:
- Re: [Wireshark-dev] [Wireshark-commits] rev 36614: /trunk/ /trunk/epan/dissectors/: packet-snmp.c /trunk/asn1/snmp/: packet-snmp-template.c
  - From: Stephen Fisher

Prev by Date: [Wireshark-dev] buildbot failure in Wireshark (development) on Visual-Studio-Code-Analysis
Next by Date: [Wireshark-dev] Short question for modus operandi
Previous by thread: Re: [Wireshark-dev] [Wireshark-commits] rev 36608: /trunk/ /trunk/epan/dissectors/: packet-x509if.c packet-x509if.h /trunk/asn1/x509if/: packet-x509if-template.c x509if.cnf
Next by thread: Re: [Wireshark-dev] [Wireshark-commits] rev 36614: /trunk/ /trunk/epan/dissectors/: packet-snmp.c /trunk/asn1/snmp/: packet-snmp-template.c
Index(es):
- Date
- Thread