Re: h.261 header parsing bug

[Jaap Keuter <jaap.keuter () xs4all nl>]

Hi,

Can you test this instead?

                /* MBAP 2nd octet, 4 bits, 3rd octet 1 bit */
                proto_tree_add_uint( h261_tree, hf_h261_mbap, tvb, offset, 1,
                    ( tvb_get_ntohs( tvb, offset ) & 0x0F80 ) >> 7 );

If this works please file a bug report in Bugzilla.

Thanks,
Jaap


On 02/28/2012 01:10 PM, Marat R. Gilmutdinov wrote:
> Hello,
>
> Looks like a bug in MBAP field parsing in H.261 header (packet-h261.c):
>
> /* MBAP 2nd octet, 4 bits, 3rd octet 1 bit */
>
> proto_tree_add_uint( h261_tree, hf_h261_mbap, tvb, offset, 1,
>
> ( tvb_get_guint8( tvb, offset ) & 15 )
>
> + ( tvb_get_guint8( tvb, offset + 1 ) >> 7 ) );
>
> offset++;
>
> ( tvb_get_guint8( tvb, offset ) & 15 ) should be multiplied by 2 (left shift by
> 1) before adding ( tvb_get_guint8( tvb, offset + 1 ) >> 7 ) ).
>
> Marat

___________________________________________________________________________
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