Wireshark mailing list archives
Re: [Wireshark-commits] rev 33464: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ber.c
From: Gerald Combs <gerald () wireshark org>
Date: Wed, 07 Jul 2010 12:14:38 -0700
Jeff Morriss wrote:
Gerald Combs wrote:Shouldn't we remove recursion from try_get_ber_length or at least throw an exception when we run into, say, 5 levels of nesting?I don't know enough to say what's a valid depth. In looking at the sample captures I have, I only got to a depth above 10 on one which is probably supposed to be valid but throws an exception in the end. But this sample capture: http://wiki.wireshark.org/SampleCaptures?action=AttachFile&do=get&target=c06-ldapv3-enc-r1.pcap.gz gets depths as high as 25,604...
That was captured from the PROTOS test suite (as was the capture I attached to bug 4951), so those depths were meant to cause problems. A search through X.690 doesn't turn up any limits on nesting. http://www.bgbm.org/TDWG/acc/Documents/asn1gloss.htm says "This nesting can be as deep or as shallow as needed." ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: [Wireshark-commits] rev 33464: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ber.c Gerald Combs (Jul 07)
- Re: [Wireshark-commits] rev 33464: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ber.c Guy Harris (Jul 07)
- Re: [Wireshark-commits] rev 33464: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ber.c Jeff Morriss (Jul 07)
- Re: [Wireshark-commits] rev 33464: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ber.c Gerald Combs (Jul 07)
- Re: [Wireshark-commits] rev 33464: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ber.c Gerald Combs (Jul 07)