summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSeungha Yang <seungha.yang@navercorp.com>2018-06-15 22:48:42 +0900
committerTim-Philipp Müller <tim@centricular.com>2018-07-18 16:39:32 +0100
commitc2b76d00e642a14aa009e06a9979217605143c1f (patch)
tree654ea1464bfdd7d75d83de6118d30341f2aadf71
parent67723a957a6a7d4c1378148588c03d54d46cf219 (diff)
downloadgstreamer-plugins-bad-c2b76d00e642a14aa009e06a9979217605143c1f.tar.gz
videoaggregator: Fix string leak
gst_video_colorimetry_to_string() returns allocated memory which must be freed. https://bugzilla.gnome.org/show_bug.cgi?id=796596
-rw-r--r--gst-libs/gst/video/gstvideoaggregator.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/gst-libs/gst/video/gstvideoaggregator.c b/gst-libs/gst/video/gstvideoaggregator.c
index 383beb39d..b155ff0a6 100644
--- a/gst-libs/gst/video/gstvideoaggregator.c
+++ b/gst-libs/gst/video/gstvideoaggregator.c
@@ -661,13 +661,14 @@ gst_video_aggregator_default_update_caps (GstVideoAggregator * vagg,
gst_caps_unref (tmp);
}
+ color_name = gst_video_colorimetry_to_string (&best_info.colorimetry);
+
GST_DEBUG_OBJECT (vagg,
"The output format will now be : %d with chroma : %s and colorimetry %s",
best_format, gst_video_chroma_to_string (best_info.chroma_site),
- gst_video_colorimetry_to_string (&best_info.colorimetry));
+ color_name);
best_format_caps = gst_caps_copy (caps);
- color_name = gst_video_colorimetry_to_string (&best_info.colorimetry);
gst_caps_set_simple (best_format_caps, "format", G_TYPE_STRING,
gst_video_format_to_string (best_format), "chroma-site", G_TYPE_STRING,
gst_video_chroma_to_string (best_info.chroma_site), "colorimetry",