Wireshark-bugs: [Wireshark-bugs] [Bug 11634] ISIS Hello trigger Error/Malformed Packet length 16
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Wed, 28 Oct 2015 07:20:42 +0000

Comment # 9 on bug 11634 from 
>> 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.