From Alexis La Goutte via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5329 :
authormorriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 22 Oct 2010 18:05:50 +0000 (18:05 +0000)
committermorriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 22 Oct 2010 18:05:50 +0000 (18:05 +0000)
commit366f753c70783cf4bee8acf0d6c1e98523127dee
tree252ad63feeb310d09bc10c1a6ada5247af8e5a8f
parentc0531218ef9ba70890564a4b4e95d9bb7f76c6d1
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.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@34624 f5534014-38df-0310-8fa8-9805f1628bb7
epan/dissectors/packet-ppp.c