Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 29535: /trunk/epan/dissectors/ /trun

From: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
Date: Mon, 24 Aug 2009 10:28:12 +0100
I'm getting this:

packet-tcp.c: In function `dissect_tcp_payload':
packet-tcp.c:1551: warning: 'cleared_writable' might be used uninitialized in this function

Not sure what the default value should be...

Martin

On Mon, Aug 24, 2009 at 10:05 AM, <etxrab@xxxxxxxxxxxxx> wrote:
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=29535

User: etxrab
Date: 2009/08/24 02:05 AM

Log:
 From: : " Michael"

 packet-tcp marks a row as unwritable if there is more
 than 1 PDU in a packet to prevent overwriting of the
 protocol column.

 However this breaks the proto,colinfo tap which checks
 rows are writable or dies EVEN if it doesn't alter anything.

 Suggested patch attached to restore the initial value
 after all PDUs have been processed.

 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3675

Directory: /trunk/epan/dissectors/
 Changes    Path            Action
 +10 -0     packet-tcp.c    Modified

___________________________________________________________________________
Sent via:    Wireshark-commits mailing list <wireshark-commits@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
            mailto:wireshark-commits-request@xxxxxxxxxxxxx?subject=unsubscribe