https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=13212
Michael Mann <mman...@netscape.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mman...@netscape.net
--- Comment #7 from Michael Mann <mman...@netscape.net> ---
(In reply to Vitaly from comment #6)
> Great.
>
> A little question:
>
> What about "length" reducing in the MS-DSH processing loop? It seams that
> the length should be reduced by 4 (the length of msi) on each step of the
> loop. It is reduced by 2 in the current code ( "length--" in the loop
> expression and "length--" in the loop statement).
>
> Something like this ( starting epan/dissectors/packet-rtcp.c:1055):
>
> while ( length >= 4 ... )
> {
> ...
> length -=4;
> }
I didn't realize how many bugs were being reported at the same time. I didn't
see any malformed packets after the first patch, so I thought I got everything.
Does the currently provided capture show this issue as well?
--
You are receiving this mail because:
You are watching all bug changes.
___________________________________________________________________________
Sent via: Wireshark-bugs mailing list <wireshark-bugs@wireshark.org>
Archives: https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe
