Comment # 6
on bug 9263
from Evan Huus
(In reply to comment #5)
> Created attachment 11773 [details]
> Don't assume tvb_[reported_]length_remaining() always return a positive
> value.
>
> With this quick patch, the following warnings are no longer present:
>
> 10:43:39 Warn Dissector bug, protocol TCP, in packet 22525:
> reassemble.c:1214: failed assertion "tvb_bytes_exist(tvb, offset,
> frag_data_len)"
> 10:43:39 Warn Dissector bug, protocol TCP, in packet 22534:
> reassemble.c:1214: failed assertion "tvb_bytes_exist(tvb, offset,
> frag_data_len)"
> 10:43:39 Warn Dissector bug, protocol TCP, in packet 25209:
> reassemble.c:1214: failed assertion "tvb_bytes_exist(tvb, offset,
> frag_data_len)"
> 10:43:39 Warn Dissector bug, protocol TCP, in packet 25321:
> reassemble.c:1214: failed assertion "tvb_bytes_exist(tvb, offset,
> frag_data_len)"
Looks good to me.
> I've wondered in the past if tvb_length_remaining() and
> tvb_reported_length_remaining() should just return 0 instead of -1. I still
> wonder.
Ya, I'd be OK with that I think. The case where the dissector wants to
distinguish between "at" and "past" the end of the buffer is rare enough that
whatever extra work they would have to do is worth the simplification here.
You are receiving this mail because:
- You are watching all bug changes.