forked from osmocom/wireshark
from Stephen fischer
Attached is a patch to fix bug #1163: "Dissector bug. ISO8073 COTP protocol." The SES dissector was incorrectly believing it had PDUs within the COTP PDUs. I added an additional heuristic check to see if the length of the SES PDU is 0, then return false since it can't be zero length. Thanks, Steve svn path=/trunk/; revision=19733
This commit is contained in:
parent
46e54a018d
commit
ed9c9cc01a
|
@ -1803,6 +1803,9 @@ dissect_ses_heur(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent_tree)
|
||||||
/* OK,let's check SPDU length */
|
/* OK,let's check SPDU length */
|
||||||
/* get length of SPDU */
|
/* get length of SPDU */
|
||||||
len = get_item_len(tvb, offset+1, &len_len);
|
len = get_item_len(tvb, offset+1, &len_len);
|
||||||
|
if(len == 0)
|
||||||
|
return FALSE; /* Not a valid PDU */
|
||||||
|
|
||||||
/* add header length */
|
/* add header length */
|
||||||
len+=len_len;
|
len+=len_len;
|
||||||
/* do we have enough bytes ? */
|
/* do we have enough bytes ? */
|
||||||
|
|
Loading…
Reference in New Issue