Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 29535:/trunk/epan/dissectors/ /trunk
From: Martin Mathieson <[email protected]>
Date: Mon, 24 Aug 2009 11:49:38 +0100
Yes thats fine now.

On Mon, Aug 24, 2009 at 10:33 AM, Anders Broman <[email protected]> wrote:

Please test with revision 29536.


From: [email protected] [mailto:[email protected]] On Behalf Of Martin Mathieson
Sent: den 24 augusti 2009 11:28
To: [email protected]
Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 29535:/trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-tcp.c

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...


On Mon, Aug 24, 2009 at 10:05 AM, <[email protected]> wrote:

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

 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.


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

Sent via:    Wireshark-commits mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
            mailto:[email protected]?subject=unsubscribe

Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:[email protected]?subject=unsubscribe