Wireshark-bugs: [Wireshark-bugs] [Bug 6237] New: packet-gsm_a_rr - misdecoding 3G Neighbour Cell

Date: Sat, 13 Aug 2011 02:52:42 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6237

           Summary: packet-gsm_a_rr - misdecoding 3G Neighbour Cell
                    Information Element in SI2quater message due to a
                    coding typo
           Product: Wireshark
           Version: 1.6.1
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Low
         Component: Wireshark
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: jqr@xxxxxxxxxxxxxx


Created an attachment (id=6824)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=6824)
diff file to fix in 1.61

Build Information:
Version 1.6.0 (SVN Rev 37592 from /trunk-1.6)

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 (64-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, without
SMI, with c-ares 1.7.1, with Lua 5.1, without Python, with GnuTLS 2.10.3, with
Gcrypt 1.4.6, without Kerberos, with GeoIP, with PortAudio V19-devel (built Jun 
7 2011), with AirPcap.

Running on 64-bit Windows 7, build 7600, 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 21022

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

Check the man page and http://www.wireshark.org for more information
--
A probable typo when copying Table 9.1.54.1a. from 3GPP TS 44.018 into the
packet-gsm_a_rr.c file has Wireshark mis-decoding the 3G Neighbour Cell
Information Element (and consequently the rest of the message) when
NR_OF_FDD_CELLS = 4 (and only 4).

>From the source code:

/*
 * [3] 10.5.2.1e Cell selection indicator after release of all TCH and SDCCH
 */
static const guint8
convert_n_to_p[32] = {   0, 10, 19, 28, 26, 44, 52, 60, 67, 74, 81, 88, 95,
102, 109, 116,
                       122,  0,  0,  0,  0,  0,  0,  0,  0,  0,  0,  0,  0,  
0,   0,   0};


>From the 3GPP literature, the element for n=4 should be 36, not 26.

The table represents the size in bits of the FDD_CELL_INFORMATION field, and if
it is incorrect, the element and the rest of the message will be incorrectly
decoded.

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