avformat/mov: parse multiple iTunes cover images
authorTimo Teräs <timo.teras@iki.fi>
Thu, 29 Mar 2018 12:59:33 +0000 (15:59 +0300)
committerMichael Niedermayer <michael@niedermayer.cc>
Sun, 1 Apr 2018 00:58:51 +0000 (02:58 +0200)
Multiple cover images are supported by having multiple data atoms
inside the covr atom. AtomicParsley and mutagen amongst others
support and document this construct.

Signed-off-by: Timo Teräs <timo.teras@iki.fi>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
libavformat/mov.c

index cb6f3a4..e7c32e6 100644 (file)
@@ -412,7 +412,11 @@ retry:
                 int ret = mov_read_covr(c, pb, data_type, str_size);
                 if (ret < 0) {
                     av_log(c->fc, AV_LOG_ERROR, "Error parsing cover art.\n");
+                    return ret;
                 }
+                atom.size -= str_size;
+                if (atom.size > 8)
+                    goto retry;
                 return ret;
             } else if (!key && c->found_hdlr_mdta && c->meta_keys) {
                 uint32_t index = AV_RB32(&atom.type);