Wireshark-bugs: [Wireshark-bugs] [Bug 2548] New: wimax UL-MAP ext IE decode error

Date: Fri, 16 May 2008 02:33:58 -0700 (PDT)
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2548

           Summary: wimax UL-MAP ext IE decode error
           Product: Wireshark
           Version: 1.0.0
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: Normal
          Priority: Medium
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: frank.wang@xxxxxxxxxxx


Build Information:
Version 1.0.99-SVN-25305 (SVN Rev 25305)

Copyright 1998-2008 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 with GTK+ 2.12.9, with GLib 2.16.3, with WinPcap (version unknown),
with libz 1.2.3, without POSIX capabilities, with libpcre 7.0, with SMI 0.4.8,
with ADNS, with Lua 5.1, with GnuTLS 1.6.1, with Gcrypt 1.2.3, with MIT
Kerberos, with PortAudio V19-devel (built May 14 2008), with AirPcap.

Running on Windows XP Service Pack 3, build 2600, with WinPcap version 4.0.2
(packet.dll version 4.0.0.1040), based on libpcap version 0.9.5, without
AirPcap.

Built using Microsoft Visual C++ 6.0 build 8804

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

When decoding UL-MAP CQICH_Alloc_IE, the decoder gived wrong result.

I found the wimax plugin used wrong MACRO in wimax_bits.h.

It uses "pletohs" instead of "pntohs". Because it is already network byte order
in UL-MAP, the pntohs should be used.

This is a commom bug that affects both the UL-MAP or DL-MAP decoder which used
"XBIT" MACRO.

I made a patch and it fixed the bug.


-- 
Configure bugmail: http://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.