Wireshark-commits: [Wireshark-commits] rev 33079: /trunk/epan/dissectors/ /trunk/epan/dissectors/:

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: wmeier@xxxxxxxxxxxxx
Date: Fri, 04 Jun 2010 00:45:39 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=33079

User: wmeier
Date: 2010/06/03 05:45 PM

Log:
 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/epan/dissectors/
  Changes    Path            Action
  +7 -8      packet-nfs.c    Modified