Linas Vepstas
changed
bug 8961
| What |
Removed |
Added |
| Attachment #11304 is obsolete |
|
1
|
Comment # 6
on bug 8961
from Linas Vepstas
Created attachment 11310 [details]
STANAG 4607 dissector patch
New improved patch; this one adds:
-- explicit check for minimum segment size, THROW's if violated
-- explicit size check for fixed-length segments. No THROW, but maybe there
should be!?
-- limits displayed tree size to 50K segments (I only ever see one segment per
packet; I can't imagine that there are other implementations that put more than
a few hundred in a packet, simply based on bandwidth and response-time
considerations.)
There are certainly many more self-consistency checks that could be made on the
data, but this becomes a big task. I'm thinking that these length checks are
enough to handle all the loss-of-framing problems ... can't think of anything
else.
You are receiving this mail because:
- You are watching all bug changes.