Wireshark-bugs: [Wireshark-bugs] [Bug 8000] New: 2 bugs in Ran-Information-Error Rim Container

Date: Mon, 19 Nov 2012 05:46:21 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8000

           Summary: 2 bugs in Ran-Information-Error Rim Container
           Product: Wireshark
           Version: 1.8.3
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Major
          Priority: Low
         Component: Wireshark
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: l.wachowski@xxxxxxxxx


Created attachment 9547
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=9547
patch created by git

Build Information:
wireshark 1.8.3 (SVN Rev Unknown from unknown)

Copyright 1998-2012 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.24.10, with Cairo 1.10.2, with Pango 1.30.0, with
GLib 2.32.3, with libpcap, with libz 1.2.3.4, without POSIX capabilities,
without SMI, without c-ares, without ADNS, without Lua, without Python, without
GnuTLS, without Gcrypt, without Kerberos, without GeoIP, without PortAudio,
with
AirPcap.

Running on Linux 3.2.0-33-generic-pae, with locale pl_PL.UTF-8, with libpcap
version 1.1.1, with libz 1.2.3.4, without AirPcap.

Built using gcc 4.6.3.
--
I have found two bugs in file packet-bssgp.c. The patch-file with my proposal
of solution you can find in attachement.

First problem is in function de_bssgp_pdu_in_error. There is used byte '0' of
message stream as a pduType, but this field in 'Pdu in error' is not the same
as pduType of the message, there should be read another byte from the stream so
I said there 'curr_offset' instead of '0'.

Second problem is in function de_bssgp_ran_inf_error_rim_cont. Error rim
container doesn't have field rim sequence number. This field exists in the rest
of containers but not in this one. So I have removed this field. 
In this function we have field pdu in error and this field is mandatory, not
optional as you have wrote there.

I have tested my solutions with using of real stream.

Best Regards

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