mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-27 12:11:13 +00:00
Code Issues 1 Projects Releases Wiki Activity

vtenc: Avoid leaking caps object and its copy

Browse source 
gst_pad_get_pad_template_caps() returns a reference which is unreferenced,
so creating a copy using gst_caps_copy() results in a reference leak. Also
the caps are pushed as an event downstream, but this doesn't consume the
caps so it must still be unreferenced.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=734534
This commit is contained in:
Sebastian Rasmussen 2014-08-09 13:20:41 +02:00 committed by Tim-Philipp Müller
parent c1d5aa3da5
commit b323fba934
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
sys/applemedia/vtenc.c
View file

@ -425,7 +425,8 @@ gst_vtenc_negotiate_downstream (GstVTEnc * self, CMSampleBufferRef sbuf)
return TRUE; return TRUE;
} }
caps = gst_caps_copy (gst_pad_get_pad_template_caps (self->srcpad)); caps = gst_pad_get_pad_template_caps (self->srcpad);
caps = gst_caps_make_writable (caps);
s = gst_caps_get_structure (caps, 0); s = gst_caps_get_structure (caps, 0);
gst_structure_set (s, gst_structure_set (s,
"width", G_TYPE_INT, self->negotiated_width, "width", G_TYPE_INT, self->negotiated_width,
@ -459,6 +460,7 @@ gst_vtenc_negotiate_downstream (GstVTEnc * self, CMSampleBufferRef sbuf)
} }
result = gst_pad_push_event (self->srcpad, gst_event_new_caps (caps)); result = gst_pad_push_event (self->srcpad, gst_event_new_caps (caps));
gst_caps_unref (caps);
self->caps_width = self->negotiated_width; self->caps_width = self->negotiated_width;
self->caps_height = self->negotiated_height; self->caps_height = self->negotiated_height;