Wireshark mailing list archives
Re: [Wireshark-commits] master-1.10 c0978c4: Don't dissect otherwise-undissected data twice.
From: Jeff Morriss <jeff.morriss.ws () gmail com>
Date: Wed, 27 May 2015 21:36:10 -0400
On 05/03/2015 06:19 PM, Wireshark code review wrote:
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c0978c46173992c87e18bfd635bef826835ad1af Submitter: Guy Harris (guy () alum mit edu) Changed: branch: master-1.10 Repository: wireshark Commits: c0978c4 by Guy Harris (guy () alum mit edu): Don't dissect otherwise-undissected data twice. Also, set decoded to TRUE after we return from a dissector; that makes it a bit clearer when we're setting it.
This change (in master-1.10) is dependent on (the packet-atm.c portion of) SVN rev 54503. Without back porting 54503 the 'decoded' variable is set-but-not-used in this branch.
Any preferences on which way to fix it? My initial suggestion would be to revert this latest change.
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: [Wireshark-commits] master-1.10 c0978c4: Don't dissect otherwise-undissected data twice. Jeff Morriss (May 27)