On 06/03/11 14:45, Holger Hans Peter Freyther wrote:
On 03/06/2011 02:20 PM, Pablo Neira Ayuso wrote:
Still, I think that I've got issue with a warning that spots a dissector
bug.
Yes, I think this was there before as well and on top of this I have never
tried the randpkt to fuzz the dissector as mentioned in doc/README.developer.
I didn't know about this, I'll check it.
From a quick look at GSM 12.21, a trace I have, it
looks like NM_ATT_SW_DESCR
is not in the TLV table and this explains why we end up here. It might be a
good opportunity to see what else we are missing.
I'll investigate this, thanks for the clue!
BTW, the output of the patch is in git format, so I think that you can
apply it with git am. It's the inner patch (which is contained inside
this patch) that it's in svn format 8-).
Yes, I am talking about git am -3 for the inner part as this (wireshark svn)
is the place where having a three way merge is going to be the most useful
thing to do. :)
Oh, I see, I didn't get it, sorry!