Wireshark-dev: Re: [Wireshark-dev] MEGACO patch is incorrect?
From: "Anders Broman" <[email protected]>
Date: Mon, 19 Jun 2006 17:43:39 +0200
Checked in.
Brg
Anders

-----Ursprungligt meddelande-----
Från: [email protected]
[mailto:[email protected]] För Tarlovskij Eugene
Skickat: den 19 juni 2006 14:25
Till: [email protected]
Ämne: [Wireshark-dev] MEGACO patch is incorrect?


Recently I've posted patch for MEGACO dissector which eliminates "Invalid
offset" error in ObservedEvents descriptor. The patch simply removes equal
sign from line 2046 of packet-megaco.c.

-			if (tvb_current_offset <= tvb_previous_offset) {
+			if (tvb_current_offset < tvb_previous_offset) {


After synchronization with wireshark repository I see that the patch is
applied in the incorrect way. The source code contains "equal" sign!

            if (tvb_current_offset = tvb_previous_offset) {

So, recent repository version still incorrect, and even more incorrect than
previous. 

I send you another patch to fix this issue.

Best,
Eugene Tarlovskij