osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

Don't use the non-existent decoded data if decode_rtp_packet() returns 0. 

Just skip that packet.  Otherwise, it crashes.

Fix file name in the introductory comment while we're at it.

Change-Id: I286f4303a4ec152c0d00c5135395c1608bf2121a
Reviewed-on: https://code.wireshark.org/review/11279
Reviewed-by: Guy Harris <guy@alum.mit.edu>
This commit is contained in:
Guy Harris 2015-10-26 03:13:08 -07:00
parent fb72cad4e8
commit 4ce9655658
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ui/qt/rtp_audio_stream.cpp
View File

@ -1,4 +1,4 @@
/* rtp_audio_frame.h
/* rtp_audio_frame.cpp
*
* Wireshark - Network traffic analyzer
* By Gerald Combs <gerald@wireshark.org>
@ -197,8 +197,9 @@ void RtpAudioStream::decode()
payload_names_ << payload_name;
}
//size_t decoded_bytes =
decode_rtp_packet(rtp_packet, &decode_buff, decoders_hash_, &channels, &sample_rate);
size_t decoded_bytes = decode_rtp_packet(rtp_packet, &decode_buff, decoders_hash_, &channels, &sample_rate);
if (decoded_bytes == 0)
continue; /* Didn't decode anything */
write_buff = (char *) decode_buff;
write_bytes = rtp_packet->info->info_payload_len * sample_bytes_;