Jaap Keuter <[EMAIL PROTECTED]> writes:

> 
> Hi,
> 
> I reworked the patch a little:
> - No C++ comments
> - No need to check for size < 11, that is already done
> - Changed the comparison to the value taken from the value_string

I've tested the change against the current svn and some captures I have and it
worked - had a bit of trouble compiling at work in my VMWare Linux install,
since gcc would ICE, but it works at home. 

The warning goes away for discover messages alright; messages not using HMAC-MD5
and delayed authentication still have 11 as the minimum size, etc. One thing
worth considering is limiting the size of the AUTH option in DHCPDISCOVER
messages to 11. 

Stefan. 

> 
> Not much changes this way. Hope you can check it out before we commit
> this. I don't have a sample capture.
> 
> Thanx,
> Jaap


_______________________________________________
Wireshark-users mailing list
[email protected]
http://www.wireshark.org/mailman/listinfo/wireshark-users

Reply via email to