Wireshark-bugs: [Wireshark-bugs] [Bug 1025] dhcp packet crashes wireshark

Date: Fri, 11 Aug 2006 18:54:52 +0000 (GMT)
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1025


ulf.lamping@xxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|0.99.0                      |0.99.2




------- Comment #6 from ulf.lamping@xxxxxx  2006-08-11 18:54 GMT -------
Hi, I've reproduced the problem and did some experiments!

The real cause is line 1209 in packet_bootp.c, and here the use of PRIx64. This
later causes trouble in the call to g_vsnprintf() in proto.c, line 2939.

I've tried several format strings, and got the following results:
- PRIx64 crashes
- PRIX64 crashes
- PRIU64 works
- llx works (GLib seems to like the GNUC variant, is this also true for GLib
1.x?)

When I remember correct, the use of these PRI... macros (defined in
config.h.win32) were needed to be used for the MSVC implementation, which
requires "I64x" in the format string in comparison to "llx" on GNUC.

However, as we no longer use the MSVC implementations, we use g_snprintf and
alike instead of snprintf (am I correct here?!?) now, we may simply no longer
need to use "I64x" and alike.

So the simplest solution might be to change the defines in config.h.nmake to
llx, or is there a reason against that I just don't see?

Regards, ULFL


-- 
Configure bugmail: http://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.