Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 25148: /trunk/epan/ /trunk/epan/: oi
From: Joerg Mayer <[email protected]>
Date: Wed, 23 Apr 2008 00:54:04 +0200
On Tue, Apr 22, 2008 at 07:31:07PM +0000, [email protected] wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=25148
> 
> User: etxrab
> Date: 2008/04/22 12:31 PM
> 
> Log:
>  Fix Dissector bug, protocol SNMP: proto.c:932: failed assertion
>  From LEGO:
>  DISSECTOR_ASSERT_NOT_REACHED() is a Bug regardless, it is triggered because we
>  are proto_add_item()ing an FT_UINT32 of 5 bytes in length.
>  
>  IF-MIB:ifInOctets is a Counter32 so the value should (and is) application
>  encoded using tag 41.
>  
>  the value's value (!) is  H'00d49e69fa (D'3567151610).
>  
>  the assertion is triggered because there's an error in oids.c:59 
>  where states the ft_type as being an FT_UINT32 with a length of 1 to 4 bytes,
>  It should be FT_UINT64 and the length from 1 to 5 bytes considering that
>  integers bigger than 2^31 will be ber encoded in 5 bytes.
>  
>  http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2468

Shouldn't the commit message name Luis as the author?

 ciao
    Joerg

-- 
Joerg Mayer                                           <[email protected]>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.