Wireshark-commits: [Wireshark-commits] rev 34624: /trunk/epan/dissectors/ /trunk/epan/dissectors/:
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: morriss@xxxxxxxxxxxxx
Date: Fri, 22 Oct 2010 18:05:53 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=34624

User: morriss
Date: 2010/10/22 11:05 AM

Log:
 From Alexis La Goutte via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5329 :
 
 Find a patch who clean up PPP dissector
 * Remove check_col (from my previous patch #5325)
 * Remove the #define ppp_min (unused) 
 * Remove some unused ett_* 
 * Cleanup NO ARRAY entry (it is a false positive of checkhf tool, it is need {
 (in the same line) before a hf_... in hf_register_info struct
 * Cleanup Unused entry in VSNP Dissector ( it's really strange, there is no
 check when the code was added #4434 ;) )
 * minor bug fix...
 
 From me: put one of the check_col()'s (that also protects a val_to_str() call)
 back in.
 
 Note: the checkhf tool bug was fixed in rev 34623.

Directory: /trunk/epan/dissectors/
  Changes    Path            Action
  +92 -189   packet-ppp.c    Modified