Comment # 7
on bug 8947
from Simon Zhong
(In reply to comment #4)
> A few other things:
>
> 1. Would it be possible to attach the trace used for fuzztesting (for future
> regression/fuzz testing)?
>
> 2. Would you be willing to help make more items filterable in the COPS
> dissector? It's a little out of scope for this bug, but the COPS dissector
> has way too many proto_tree_add_text calls. There is a tool for converting
> them (convert_proto_tree_add_text.pl), but having someone more familiar with
> the protocol (which I'm not) helps for variable and display filter naming.
Thanks Michael.
1. I will upload the capture file I used in fuzz test after I made the change
which Evan suggested (and tested again).
2. I'm aware of the proto_tree_add_text issue in packet-cops.c, I've ran
checkAPIs.pl before I open this ticket and got warning of 69 useless
add_test(). I was just not sure if I should fix them because they were there
already :) I will be happy to do the cleanup, would you suggest I deliver them
in this patch or open another one? IMO a dedicate patch would be more clear.
You are receiving this mail because:
- You are watching all bug changes.