[vlc-devel] [PATCH 1/1] opus: handle RTP depayloading (fixes #11938)

Ross Finlayson finlayson at live555.com
Sun Jan 11 20:53:54 CET 2015

> IMHO, live555 should reject SDP, or at least rtpmap, with nul clock rate as that makes no sense. Otherwise, VLC should skip those.

FYI, should the LIVE555 library sees a SDP description with a zero (or missing) RTP timestamp frequency, it will set it to some sane non-zero value.  So VLC's code should never need to check for this.

In particular, you can remove the test
	if ( tk->fmt.audio.i_rate == 0 )
from line 979 (the 'Speex' branch) of "live555.cpp", because that will never happen.

Ross Finlayson
Live Networks, Inc.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20150111/73132a28/attachment.html>

More information about the vlc-devel mailing list