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-mount.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-mount.c')
-rw-r--r-- | epan/dissectors/packet-mount.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/dissectors/packet-mount.c b/epan/dissectors/packet-mount.c index e7318ae495..5866c63152 100644 --- a/epan/dissectors/packet-mount.c +++ b/epan/dissectors/packet-mount.c @@ -174,7 +174,8 @@ dissect_mount_dirpath_call(tvbuff_t *tvb, int offset, packet_info *pinfo, host=ip_to_str(pinfo->dst.data); len=tvb_get_ntohl(tvb, offset); - DISSECTOR_ASSERT(len < ITEM_LABEL_LENGTH); + if (len >= ITEM_LABEL_LENGTH) + THROW(ReportedBoundsError); dir=tvb_get_ptr(tvb, offset+4, len); if(dir){ |