Wireshark-commits: [Wireshark-commits] rev 33133: /trunk-1.2/epan/dissectors/ /trunk-1.2/epan/disse

Date: Mon, 07 Jun 2010 16:07:28 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=33133

User: gerald
Date: 2010/06/07 09:07 AM

Log:
 Copy over r33079:
 
 ------------------------------------------------------------------------
 r33079 | wmeier | 2010-06-03 17:45:37 -0700 (Thu, 03 Jun 2010) | 12 lines
 Changed paths:
    M /trunk/epan/dissectors/packet-nfs.c
 
 From Cal Turney: The NFS FH hash (nfs.fh.hash) incorrectly matches multiple filehandles.
 "The method used in packet-nfs.c to calculate a 32-bit hash representing the 
  32-byte filehandle is faulty in that the hash often matches multiple
  filehandles."
 
 "This patch uses CRC-32 to calculate the hash.
  We (EMC GNS) have tested this patch for the past two years and we have not
  found a single case where the hash matched more than one filehandle."
 
 See Bug #4839: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4839
 ------------------------------------------------------------------------

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