Since track->audio.bitdepth is zero for A_QUICKTIME, the sample size has to be retrieved from the private data. Also, 'twos' and 'sowt' audio can be signed 8-bit.

Mats
>From dbddf5d5fba7992e52bb25db214e7b4debfb8023 Mon Sep 17 00:00:00 2001
From: Mats Peterson <matsp...@yahoo.com>
Date: Fri, 15 Jan 2016 23:25:38 +0100
Subject: [PATCH] lavf/matroskadec: Get sample size from private data

---
 libavformat/matroskadec.c |   10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
index cc5ec19..d788232 100644
--- a/libavformat/matroskadec.c
+++ b/libavformat/matroskadec.c
@@ -1891,18 +1891,24 @@ static int matroska_parse_tracks(AVFormatContext *s)
                    /* Normally 36, but allow noncompliant private data */
                    && (track->codec_priv.size >= 32)
                    && (track->codec_priv.data)) {
+            uint16_t sample_size;
             int ret = get_qt_codec(track, &fourcc, &codec_id);
             if (ret < 0)
                 return ret;
+            sample_size = AV_RB16(track->codec_priv.data + 26);
             if (fourcc == 0) {
-                if (track->audio.bitdepth == 8) {
+                if (sample_size == 8) {
                     fourcc = MKTAG('r','a','w',' ');
                     codec_id = ff_codec_get_id(ff_codec_movaudio_tags, fourcc);
-                } else if (track->audio.bitdepth == 16) {
+                } else if (sample_size == 16) {
                     fourcc = MKTAG('t','w','o','s');
                     codec_id = ff_codec_get_id(ff_codec_movaudio_tags, fourcc);
                 }
             }
+            if ((fourcc == MKTAG('t','w','o','s') ||
+                    fourcc == MKTAG('s','o','w','t')) &&
+                    sample_size == 8)
+                codec_id = AV_CODEC_ID_PCM_S8;
         } else if (!strcmp(track->codec_id, "V_QUICKTIME") &&
                    (track->codec_priv.size >= 21)          &&
                    (track->codec_priv.data)) {
-- 
1.7.10.4

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to