Comment # 9
on bug 11634
from Jim Young
>> pdu_length = tvb_get_ntohs(tvb, 9) - 8;
> I still kinda like this based on the options (passing packet length seems painful
> to all of the other options that don't care), would also probably have to adjust
> dissect_snp_checksum_clv()
With my initial proposed fix we no longer see an Error/Malformed message, But
we have a new problem; the ISIS Hello checksum now fails. In the attached
sample trace the ISIS Hellos' PDU length fields are reported as 173 octets
long, but we only pass in the last 165 octets; we need those 8 previous octets
if we really want to validate CRC. (That's why the the original magic offset
value was 17.)
I need to take another look later after I've had some sleep.
You are receiving this mail because:
- You are watching all bug changes.