Wireshark mailing list archives
Re: Rename TVB captured length vs reported length
From: Dirk Jagdmann <doj () cubic org>
Date: Tue, 18 Feb 2014 14:00:31 -0800
You *can* continue dissecting the protocol. You just get an exception, so that the user sees that the data was in the packet, but the snapshot length was set too short to capture it. Stopping dissection just because the data to be dissected was cut off by a snapshot length is entirely the wrong thing to do, as it can make it look as if the packet didn't have the data in question.
That's my point. Not all of the code I have written will get executed, because data is missing. And if the next frame needs information from previous frames to dissect, my dissection of the protocol stops here. Of course there's nothing I can do about it. I just wanted to point out, that not only 5% is the dissector code requires access to all data to be useful. My main point is however to not use "tvb_length" any more at all, because different developers will interpret it differently. Use the longer, but explicit function names. -- ---> Dirk Jagdmann ----> http://cubic.org/~doj -----> http://llg.cubic.org ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://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: Rename TVB captured length vs reported length, (continued)
- Re: Rename TVB captured length vs reported length Guy Harris (Feb 17)
- Re: Rename TVB captured length vs reported length Guy Harris (Feb 17)
- Re: Rename TVB captured length vs reported length Jakub Zawadzki (Feb 17)
- Re: Rename TVB captured length vs reported length Jeff Morriss (Feb 17)
- Re: Rename TVB captured length vs reported length Gilbert Ramirez (Feb 17)
- Re: Rename TVB captured length vs reported length Guy Harris (Feb 17)
- Re: Rename TVB captured length vs reported length Jeff Morriss (Feb 17)
- Re: Rename TVB captured length vs reported length Hadriel Kaplan (Feb 17)
- Re: Rename TVB captured length vs reported length Alexis La Goutte (Feb 18)
- Re: Rename TVB captured length vs reported length Dirk Jagdmann (Feb 18)
- Re: Rename TVB captured length vs reported length Guy Harris (Feb 18)
- Re: Rename TVB captured length vs reported length Dirk Jagdmann (Feb 18)
- Re: Rename TVB captured length vs reported length Guy Harris (Feb 18)
- Re: Rename TVB captured length vs reported length Guy Harris (Feb 18)
- Re: Rename TVB captured length vs reported length Hadriel Kaplan (Feb 18)
- Re: Rename TVB captured length vs reported length Guy Harris (Feb 17)