From ecac298e4e53df93beb11622c9def98c1e978ba9 Mon Sep 17 00:00:00 2001 From: Wim Taymans Date: Mon, 13 Jan 2014 16:29:00 +0100 Subject: [PATCH] videodecoder: only copy chroma_site when known Only overwrite the chroma-site if we have a valid value in the reference format. --- gst-libs/gst/video/gstvideodecoder.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/gst-libs/gst/video/gstvideodecoder.c b/gst-libs/gst/video/gstvideodecoder.c index a1533d0d7b..6f4250605b 100644 --- a/gst-libs/gst/video/gstvideodecoder.c +++ b/gst-libs/gst/video/gstvideodecoder.c @@ -732,9 +732,10 @@ _new_output_state (GstVideoFormat fmt, guint width, guint height, /* Copy over extra fields from reference state */ tgt->interlace_mode = ref->interlace_mode; tgt->flags = ref->flags; - tgt->chroma_site = ref->chroma_site; /* only copy values that are not unknown so that we don't override the * defaults. subclasses should really fill these in when they know. */ + if (ref->chroma_site) + tgt->chroma_site = ref->chroma_site; if (ref->colorimetry.range) tgt->colorimetry.range = ref->colorimetry.range; if (ref->colorimetry.matrix)