rtp: Fix ff_rtp_get_payload_type
authorMartin Storsjö <martin@martin.st>
Mon, 17 Oct 2011 17:24:54 +0000 (20:24 +0300)
committerMartin Storsjö <martin@martin.st>
Mon, 17 Oct 2011 17:40:20 +0000 (20:40 +0300)
It was broken in 3b3ea34655db02d9cd9ea1a4122e920a7fdec602
"Remove all uses of deprecated AVOptions API", where any
presence of a payload_type AVOption caused its value to
be returned, even if it wasn't set (and thus had the default
-1 value).

This caused the RTP muxer to be broken.

Signed-off-by: Martin Storsjö <martin@martin.st>
libavformat/rtp.c

index 4be845a..b6b4b72 100644 (file)
@@ -98,7 +98,8 @@ int ff_rtp_get_payload_type(AVFormatContext *fmt, AVCodecContext *codec)
     /* Was the payload type already specified for the RTP muxer? */
     if (ofmt && ofmt->priv_class) {
         int64_t payload_type;
-        if (av_opt_get_int(fmt->priv_data, "payload_type", 0, &payload_type) >= 0)
+        if (av_opt_get_int(fmt->priv_data, "payload_type", 0, &payload_type) >= 0 &&
+            payload_type >= 0)
             return (int)payload_type;
     }