diff options
| author | Gerald Combs <gerald@wireshark.org> | 2006-07-09 22:12:02 +0000 |
|---|---|---|
| committer | Gerald Combs <gerald@wireshark.org> | 2006-07-09 22:12:02 +0000 |
| commit | 8b315480f1df953751fda12c58ed891862e03edc (patch) | |
| tree | 55cf0cb616310c4e42a93835a86f9ecffedaad3e /epan/dissectors/packet-pktc.c | |
| parent | 6dd2196da600c33b59572c1571f4f7dc3225c870 (diff) | |
In new_format_oid(), handle a zero-length OID.
Fix a bug introduced recently in packet-rpc.c.
Replace DISSECTOR_ASSERT() with THROW(ReportedBoundsError) in my recent
checkins, since fuzz-test.sh sets WIRESHARK_ABORT_ON_DISSECTOR_BUG.
svn path=/trunk/; revision=18693
Diffstat (limited to 'epan/dissectors/packet-pktc.c')
| -rw-r--r-- | epan/dissectors/packet-pktc.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/dissectors/packet-pktc.c b/epan/dissectors/packet-pktc.c index ba865e980b..2e43177f65 100644 --- a/epan/dissectors/packet-pktc.c +++ b/epan/dissectors/packet-pktc.c @@ -521,7 +521,8 @@ dissect_pktc_mtafqdn_krbsafeuserdata(packet_info *pinfo, tvbuff_t *tvb, proto_tr case PKTC_MTAFQDN_REP: /* MTA FQDN */ string_len = tvb_length_remaining(tvb, offset) - 4; - DISSECTOR_ASSERT(string_len > 0); + if (string_len <= 0) + THROW(ReportedBoundsError); proto_tree_add_item(tree, hf_pktc_mtafqdn_fqdn, tvb, offset, string_len, FALSE); offset+=string_len; |