Re: Patch for rtp_analysis.c: delta_timestamp problem
Joerg Mayer <[email protected]> Thu, 27 Jul 2006 16:21:49 +0200
| Newsgroups | gmane.network.ethereal.devel |
|---|---|
| Message-ID | <[email protected]> |
On Tue, Jul 18, 2006 at 04:39:35PM +0300, Alex Shnitman wrote: > Below is a patch for a bug in rtp_analysis.c. The variable statinfo->delta_timestamp was not initialized in all the necessary cases. The function rtp_packet_save_payload uses this variable and the for loop on line 935 could use an old value and run an incorrect amount of times, e.g. a couple billion, which gets ethereal stuck. I can supply an example capture which causes it. The patch solves the problem. Committed revision 18798. Thanks for your contribution! Please note though, that your patch was not applyed to the Ethereal sources but to the Wireshark sources as the Ethereal project has been renamed. Please send further mail to [email protected] and have a look at www.wireshark.org to download a current snapshot of the sources including your patch. 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.