ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-dev: Re: [Wireshark-dev] [resend]MMS dissector bug?

From: "ronnie sahlberg" <ronniesahlberg@xxxxxxxxx>
Date: Tue, 21 Nov 2006 17:42:46 +1100
what needs to be done is likely to check the length of the encoded bitstring in the ber encoding and abort once all the encoded bits have been consumed instead of continuing until the end of the ASN definition.



On 11/21/06, Anders Broman <a.broman@xxxxxxxxx> wrote:
Hi,
That section was added on request. I have forgotten the reason.
But I think it was discussed on the mailing list.

The original ASN1 can be found at:
http://www.sisconet.com/downloads/mms_abstract_syntax.txt

BR
Anders
P.S
The check in message was:

Revision 18974 - (view) (download) (as text) - [select for diffs]
Modified Mon Aug 21 13:32:07 2006 UTC (3 months ago) by sahlberg
File length: 67109 byte(s)
Diff to previous 18214
from Angel de Juan

make mms bitstrings easier to read


--
Hi, this is my new mms with some changes in order to display some data in
binary instead of hex.

Excuse me because is not a diff from the repository, but I don't know how to
create a diff.

I also have the compiled packet-mms.c and packet-mms.h if you want it only
mail me.


-----Ursprungligt meddelande-----
Från: wireshark-dev-bounces@xxxxxxxxxxxxx
[mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] För Stephen Fisher
Skickat: den 21 november 2006 00:40
Till: Steven.Schaeffer@xxxxxxx; Developer support list for Wireshark
Ämne: Re: [Wireshark-dev] [resend]MMS dissector bug?

On Mon, Nov 20, 2006 at 11:59:32AM -0700, Steve Schaeffer wrote:

> It seems to me that an 8-bit "bit-string" should only show 8 entries
> in the packet details, not 512 as in packet #497 in the attachment to
> bug #1229:
>                         success: bit-string (4)
>                             Padding: 3
>                             bit-string: 80 (offset0)
>                                 1... .... = offset0: True

>                                 .... ...0 = offset511: False

I'm not familiar with the protocol, but it looks like all 512 are hard
coded into the dissector.  Does the bit-string need to be only 8 bits
long?  Do you have access to the standards documents by chance?


Steve

_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev

_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev