[libav-stable] ape: Don't allow the seektable to be omitted

Martin Storsjö git at libav.org
Thu Sep 12 09:58:53 CEST 2013


Module: libav
Branch: master
Commit: 183b9d843a9533774fabd3984a52f3987001acbc

Author:    Martin Storsjö <martin at martin.st>
Committer: Martin Storsjö <martin at martin.st>
Date:      Wed Sep 11 22:29:33 2013 +0300

ape: Don't allow the seektable to be omitted

The seektable is required for filling in ape->frames[i].pos
further down.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable at libav.org
Signed-off-by: Martin Storsjö <martin at martin.st>

---

 libavformat/ape.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/libavformat/ape.c b/libavformat/ape.c
index 4079947..d967a5d 100644
--- a/libavformat/ape.c
+++ b/libavformat/ape.c
@@ -260,7 +260,7 @@ static int ape_read_header(AVFormatContext * s)
                ape->totalframes);
         return -1;
     }
-    if (ape->seektablelength && (ape->seektablelength / sizeof(*ape->seektable)) < ape->totalframes) {
+    if (ape->seektablelength / sizeof(*ape->seektable) < ape->totalframes) {
         av_log(s, AV_LOG_ERROR,
                "Number of seek entries is less than number of frames: %zu vs. %"PRIu32"\n",
                ape->seektablelength / sizeof(*ape->seektable), ape->totalframes);



More information about the libav-stable mailing list