[FFmpeg-devel] [PATCH] rmenc: more than 2 streams are not currently possible.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Apr 29 00:14:47 CEST 2012


It is possible that just extending the RMMuxContext.streams
array would avoid it.
It is also possible that two audio streams will fail to mux
correctly as well, though at least it should not crash for
this reason.
I do not feel like checking either of these.
This patch fixes trac issue #1022 (at least it makes it
exit with a proper error message instead of crashing).

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
---
 libavformat/rmenc.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/libavformat/rmenc.c b/libavformat/rmenc.c
index 8a9d840..d6a50ea 100644
--- a/libavformat/rmenc.c
+++ b/libavformat/rmenc.c
@@ -309,6 +309,11 @@ static int rm_write_header(AVFormatContext *s)
     int n;
     AVCodecContext *codec;
 
+    if (s->nb_streams > 2) {
+        av_log(s, AV_LOG_ERROR, "At most 2 streams are currently supported for muxing in RM\n");
+        return -1;
+    }
+
     for(n=0;n<s->nb_streams;n++) {
         s->streams[n]->id = n;
         codec = s->streams[n]->codec;
-- 
1.7.10



More information about the ffmpeg-devel mailing list