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

Hi and thanks for having a look. I'm not sure I understand item 3 - do you 
mean comparing with opt53_text[1] instead of "Discover"? 

I have some sample captures to test with, unfortunately I've set my newsgroup
membership so that I don't get any emails and use gmane.org for reading and
posting, and it seems I can't attach files with that; I can provide them to 
you if you care. 

Anyway, I'll build wireshark today with the changes below and let you know the
results. 

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
Wireshark-users@wireshark.org
http://www.wireshark.org/mailman/listinfo/wireshark-users

Reply via email to