Wireshark-commits: [Wireshark-commits] rev 34115: /trunk/epan/dissectors/ /trunk/epan/dissectors/:
Date: Tue, 14 Sep 2010 15:43:28 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=34115

User: gerald
Date: 2010/09/14 08:43 AM

Log:
 From Cal Turney via bug 5209:
 
 Decode of SETCLIENTID calls in the Windows x86 version fail with "[Dissector
 bug, protocol NFS: STATUS_ACCESS_VIOLATION: dissector accessed an invalid
 memory address]".  This error occurs in packet-nfs.c in
 dissect_nfs_clientaddr4() where vars 'protocol' and 'universal_ip_address' get
 stepped on following the call to scanf().  The b1-b10 vars are declared as
 quint8.  While "hh" modifier used in the scanf() is documented in Linux to
 correspond to an a signed/unsigned char arg, I cannot find a similar
 designation in Windows (MSDN). The Windows C compiler interprets %hhu as
 corresponding to a int16 rather than int8.

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