]> git.sesse.net Git - ffmpeg/commitdiff
lavc: make up a fake frame channel layout when there is no real one.
authorAnton Khirnov <anton@khirnov.net>
Mon, 11 Feb 2013 07:13:22 +0000 (08:13 +0100)
committerAnton Khirnov <anton@khirnov.net>
Fri, 8 Mar 2013 06:40:06 +0000 (07:40 +0100)
This should ensure that a valid channel layout is always set on a frame,
until a better solution is implemented.

libavcodec/utils.c

index 90f25377f6336d02fa36947cf5f182b59078a38f..62fc1b56b1467e3b1825784c457c784bfcba113f 100644 (file)
@@ -569,8 +569,28 @@ int ff_get_buffer(AVCodecContext *avctx, AVFrame *frame, int flags)
             frame->sample_rate    = avctx->sample_rate;
         if (frame->format < 0)
             frame->format         = avctx->sample_fmt;
-        if (!frame->channel_layout)
-            frame->channel_layout = avctx->channel_layout;
+        if (!frame->channel_layout) {
+            if (avctx->channel_layout) {
+                 if (av_get_channel_layout_nb_channels(avctx->channel_layout) !=
+                     avctx->channels) {
+                     av_log(avctx, AV_LOG_ERROR, "Inconsistent channel "
+                            "configuration.\n");
+                     return AVERROR(EINVAL);
+                 }
+
+                frame->channel_layout = avctx->channel_layout;
+            } else {
+                if (avctx->channels > FF_SANE_NB_CHANNELS) {
+                    av_log(avctx, AV_LOG_ERROR, "Too many channels: %d.\n",
+                           avctx->channels);
+                    return AVERROR(ENOSYS);
+                }
+
+                frame->channel_layout = av_get_default_channel_layout(avctx->channels);
+                if (!frame->channel_layout)
+                    frame->channel_layout = (1ULL << avctx->channels) - 1;
+            }
+        }
         break;
     default: return AVERROR(EINVAL);
     }