shorten: skip some fields in the WAV header embedded in the shorten header.
authorJustin Ruggles <justin.ruggles@gmail.com>
Wed, 14 Sep 2011 19:20:22 +0000 (15:20 -0400)
committerJustin Ruggles <justin.ruggles@gmail.com>
Thu, 20 Oct 2011 17:09:25 +0000 (13:09 -0400)
fixes incorrect bitrate reporting and potential misreporting of the number of
channels.

libavcodec/shorten.c

index af12e08..af798da 100644 (file)
@@ -231,10 +231,10 @@ static int decode_wave_header(AVCodecContext *avctx, uint8_t *header, int header
             return -1;
     }
 
             return -1;
     }
 
-    avctx->channels = get_le16(&hb);
+    skip_bits(&hb, 16); // skip channels    (already got from shorten header)
     avctx->sample_rate = get_le32(&hb);
     avctx->sample_rate = get_le32(&hb);
-    avctx->bit_rate = get_le32(&hb) * 8;
-    avctx->block_align = get_le16(&hb);
+    skip_bits(&hb, 32); // skip bit rate    (represents original uncompressed bit rate)
+    skip_bits(&hb, 16); // skip block align (not needed)
     avctx->bits_per_coded_sample = get_le16(&hb);
 
     if (avctx->bits_per_coded_sample != 16) {
     avctx->bits_per_coded_sample = get_le16(&hb);
 
     if (avctx->bits_per_coded_sample != 16) {
@@ -295,6 +295,7 @@ static int read_header(ShortenContext *s)
         av_log(s->avctx, AV_LOG_ERROR, "too many channels: %d\n", s->channels);
         return -1;
     }
         av_log(s->avctx, AV_LOG_ERROR, "too many channels: %d\n", s->channels);
         return -1;
     }
+    s->avctx->channels = s->channels;
 
     /* get blocksize if version > 0 */
     if (s->version > 0) {
 
     /* get blocksize if version > 0 */
     if (s->version > 0) {