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

Wireshark-bugs: [Wireshark-bugs] [Bug 5209] Decode of SETCLIENTID calls fails with STATUS_ACCESS

Date: Tue, 14 Sep 2010 07:23:24 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5209

Cal Turney <turney_cal@xxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |turney_cal@xxxxxxx

--- Comment #4 from Cal Turney <turney_cal@xxxxxxx> 2010-09-14 07:23:21 PDT ---
(In reply to comment #3)
> It seems we should rule out "hh" format specifier since it's C99, see:
> http://connect.microsoft.com/VisualStudio/feedback/details/416843/sscanf-cannot-not-handle-hhd-format
> and
> http://www.opengroup.org/onlinepubs/009695399/functions/scanf.html

Good catch.  This article explains why converting to guint16 solved the
problem. I tested with 'hh' removed throughout the routine and the IPv4 path
works fine.  I am unable to test the last two paths because I don't have
captures that match these scenarios but I they should work OK as well.  

I've uploaded the updated path.

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