Wireshark mailing list archives
Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c
From: Alexander Koeppe <format_c () online de>
Date: Fri, 12 Nov 2010 14:14:56 +0100
Jakub Zawadzki schrieb:
On Fri, Nov 12, 2010 at 12:26:39PM +0100, Alexander Koeppe wrote:As Jakub didn't react, I opened a bug# 5383 for the patch on behalf.Btw. should get_byte_view_data_and_length() return tvb_reported_length()?I'm still waiting for response from core developers ;) In your patch you didn't change get_byte_view_data_and_length, it works ok?
Yes it does. I just undid the patches from bug# 5285 and 5374 and tested with the faulty pcap provided in bug# 5285. It worked as expected. bend execeeded len when selecting the last PDU and has been limited to len by the check you provided. Thus Gtk highlights only the bytes actually available and has no need to crash anymore. So why do you think it's necessary to modify get_byte_view_data_and_length() for this? Cheers ___________________________________________________________________________ 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: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Stig Bjørlykke (Nov 09)
- Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Jaap Keuter (Nov 09)
- Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Jakub Zawadzki (Nov 09)
- Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Alexander Koeppe (Nov 10)
- Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Alexander Koeppe (Nov 12)
- Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Jakub Zawadzki (Nov 12)
- Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Alexander Koeppe (Nov 12)
- Re: [Wireshark-commits] rev 34796: /trunk/gtk/ /trunk/gtk/: main_proto_draw.c Alexander Koeppe (Nov 10)