Wireshark-bugs: [Wireshark-bugs] [Bug 6087] New: [ASN.1 PER] Do not trigger an assert when reach
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Sun, 3 Jul 2011 12:07:08 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6087

           Summary: [ASN.1 PER] Do not trigger an assert when reaching an
                    unknown extension in CHOICE and SEQUENCE types
           Product: Wireshark
           Version: SVN
          Platform: Other
        OS/Version: Windows Vista
            Status: NEW
          Severity: Normal
          Priority: Low
         Component: Wireshark
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: pascal.quantin@xxxxxxxxx


Created an attachment (id=6607)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=6607)
replace assert on unknown extension by an expert info

Build Information:
Version 1.7.0 (SVN Rev 37872 from /trunk)

Copyright 1998-2011 Gerald Combs <gerald@xxxxxxxxxxxxx> and contributors.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiled (32-bit) with GTK+ 2.22.1, with GLib 2.26.1, with WinPcap (version
unknown), with libz 1.2.5, without POSIX capabilities, without libpcre, with
SMI
0.4.8, with c-ares 1.7.1, with Lua 5.1, without Python, with GnuTLS 2.10.3,
with
Gcrypt 1.4.6, with MIT Kerberos, with GeoIP, with PortAudio V19-devel (built
Jul
 3 2011), with AirPcap.

Running on Windows XP Service Pack 3, build 2600, with WinPcap version 4.1.2
(packet.dll version 4.1.0.2001), based on libpcap version 1.0 branch 1_0_rel0b
(20091008), GnuTLS 2.10.3, Gcrypt 1.4.6, without AirPcap.

Built using Microsoft Visual C++ 9.0 build 30729

Wireshark is Open Source Software released under the GNU General Public
License.

Check the man page and http://www.wireshark.org for more information.
--
Currently, when the ASN.1 PER decoder encounters an unknown extension marker,
it calls the macro PER_NOT_DECODED_YET. This macro display some text and make a
call to tvb_get_guint8(tvb, 9999) so as to trigger an assert.
Some (all ?) protocols likre LTE RRC rely on those extension markers to add
extensions in future releases while keeping backward compatibility.
The attached patch proposes to replace the call to PER_NOT_DECODED_YET with an
expert information PI_UNDECODED of level PI_NOTE when the dissector finds an
unknown extension marker in CHOICE or SEQUENCE types. It allows to decode the
message according to the known ASN.1 description while notifying the user that
some part of the message could not be decoded because it is unknown.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.