From 0276b9524294e518cdc7cbfa12b7cb301ed86fb6 Mon Sep 17 00:00:00 2001 From: Jochen Strunk Date: Tue, 24 Mar 2015 10:39:27 +0100 Subject: [PATCH] avformat/mov: check color parameter type explicitly. Fixes ticket #4387. Signed-off-by: Michael Niedermayer Note, Vittorio Giovara had submitted a very similar fix to me privately a few hours before this, iam applying Jochens because it comes with a commit message too and i had not yet applied Vittorios, but For sake of credit, Vittorio independently solved this first --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 1f57dab89a..dc31e5a6f7 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1001,7 +1001,7 @@ static int mov_read_colr(MOVContext *c, AVIOContext *pb, MOVAtom atom) av_dlog(c->fc, "%s: pri %d trc %d matrix %d", color_parameter_type, color_primaries, color_trc, color_matrix); - if (c->isom) { + if (strncmp(color_parameter_type, "nclx", 4) == 0) { uint8_t color_range = avio_r8(pb) >> 7; av_dlog(c->fc, " full %"PRIu8"", color_range); if (color_range)