[FFmpeg-cvslog] pcmdec: move DVD PCM bits_per_coded_sample check near to the code that sets

Justin Ruggles git at videolan.org
Thu Oct 27 01:46:44 CEST 2011


ffmpeg | branch: master | Justin Ruggles <justin.ruggles at gmail.com> | Tue Sep 27 19:24:47 2011 -0400| [381e195b46d080aee1d9b05ef2b6b140e9463519] | committer: Justin Ruggles

pcmdec: move DVD PCM bits_per_coded_sample check near to the code that sets
the sample size.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=381e195b46d080aee1d9b05ef2b6b140e9463519
---

 libavcodec/pcm.c |   12 +++++++-----
 1 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/libavcodec/pcm.c b/libavcodec/pcm.c
index 4b11583..158366c 100644
--- a/libavcodec/pcm.c
+++ b/libavcodec/pcm.c
@@ -279,10 +279,15 @@ static int pcm_decode_frame(AVCodecContext *avctx,
     sample_size = av_get_bits_per_sample(avctx->codec_id)/8;
 
     /* av_get_bits_per_sample returns 0 for CODEC_ID_PCM_DVD */
-    if (CODEC_ID_PCM_DVD == avctx->codec_id)
+    if (CODEC_ID_PCM_DVD == avctx->codec_id) {
+        if (avctx->bits_per_coded_sample != 20 &&
+            avctx->bits_per_coded_sample != 24) {
+            av_log(avctx, AV_LOG_ERROR, "PCM DVD unsupported sample depth\n");
+            return AVERROR(EINVAL);
+        }
         /* 2 samples are interleaved per block in PCM_DVD */
         sample_size = avctx->bits_per_coded_sample * 2 / 8;
-    else if (avctx->codec_id == CODEC_ID_PCM_LXF)
+    } else if (avctx->codec_id == CODEC_ID_PCM_LXF)
         /* we process 40-bit blocks per channel for LXF */
         sample_size = 5;
 
@@ -433,9 +438,6 @@ static int pcm_decode_frame(AVCodecContext *avctx,
                 src = src8;
             }
             break;
-        default:
-            av_log(avctx, AV_LOG_ERROR, "PCM DVD unsupported sample depth\n");
-            return -1;
         }
         samples = (uint8_t *) dst_int32_t;
         break;



More information about the ffmpeg-cvslog mailing list