Wireshark · Wireshark-users: Re: [Wireshark-users] TCP Decoding differences between Ethereal0.99and Wireshark 0.99.3/4?

["Small, James" <JSmall@xxxxxxxxxxxxxx>]

Excellent!

So if I download and install SVN-19778:
http://www.wireshark.org/download/automated/win32/wireshark-setup-0.99.4
-SVN-19778.exe

It will have this fix?

--Jim


-----Original Message-----
From: wireshark-users-bounces@xxxxxxxxxxxxx
[mailto:wireshark-users-bounces@xxxxxxxxxxxxx] On Behalf Of Joerg Mayer
Sent: Thursday, November 02, 2006 3:57 AM
To: Community support list for Wireshark
Subject: Re: [Wireshark-users] TCP Decoding differences between
Ethereal0.99and Wireshark 0.99.3/4?

On Thu, Nov 02, 2006 at 07:38:41AM +0100, Jaap Keuter wrote:
> On Wed, 1 Nov 2006, Small, James wrote:
> > Jaap/Ulf - I know you are busy and this does not appear to be a high
> > priority bug.  Is there any work around to "disable" the bug 852 fix
so
> > that if you want to easily display TCP stream text and are willing
to
> > except the crash risk you can?

> Use the undo_fix_852.patch to, well, undo the the fix.

I've just committed 19773, which fixes this bug. As the fix is rather
trivial I'd like someone who knows that code to have a look at it and
verify that it doesn't have any negative side effects before I close the
bug.

Here it comes:

Index: follow_dlg.c
===================================================================
--- follow_dlg.c        (revision 19771)
+++ follow_dlg.c        (working copy)
@@ -607,7 +607,7 @@
     gchar               initbuf[256];
     guint32             server_packet_count = 0;
     guint32             client_packet_count = 0;
-    char                buffer[FLT_BUF_SIZE];
+    char                buffer[FLT_BUF_SIZE+1]; /* +1 to fix ws bug
1043 */
     size_t              nchars;
     static const gchar hexchars[16] = "0123456789abcdef";

@@ -1010,7 +1010,7 @@
      * doesn't blow up on excessive long lines.
      */
     if (line_break == FALSE)
-        buffer[--i] = '\n';
+        buffer[i] = '\n';
 #endif

 #if GTK_MAJOR_VERSION < 2

Please let me know whether the fix is OK so I may close the bug (I know
that
it fixes the sample capture attached to 1043).

Ciao
    Joerg

-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
_______________________________________________
Wireshark-users mailing list
Wireshark-users@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-users