After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 748353 - rtspsrc: Leak of RTCP caps
rtspsrc: Leak of RTCP caps
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 1.5.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-04-23 08:04 UTC by Jesper Larsen
Modified: 2015-04-23 14:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
rtspsrc: Fix RTCP caps leak (697 bytes, patch)
2015-04-23 08:04 UTC, Jesper Larsen
committed Details | Review

Description Jesper Larsen 2015-04-23 08:04:34 UTC
Created attachment 302204 [details] [review]
rtspsrc: Fix RTCP caps leak

Running a pipeline containing rtspsrc through Valgrind gave me this

==11275== 200 (72 direct, 128 indirect) bytes in 1 blocks are definitely lost in loss record 4,270 of 4,457
==11275==    at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==11275==    by 0x53EB560: g_malloc (gmem.c:97)
==11275==    by 0x540123D: g_slice_alloc (gslice.c:1007)
==11275==    by 0x4E7710E: gst_caps_new_empty (gstcaps.c:243)
==11275==    by 0x4E7721D: gst_caps_new_empty_simple (gstcaps.c:288)
==11275==    by 0x73E22FD: gst_rtspsrc_handle_data (gstrtspsrc.c:4632)
==11275==    by 0x73EDFDE: gst_rtspsrc_thread (gstrtspsrc.c:4799)
==11275==    by 0x4ED0048: gst_task_func (gsttask.c:331)
==11275==    by 0x540B9BB: g_thread_pool_thread_proxy (gthreadpool.c:307)
==11275==    by 0x540B034: g_thread_proxy (gthread.c:764)
==11275==    by 0x56DB181: start_thread (pthread_create.c:312)
==11275==    by 0x59EB47C: clone (clone.S:111)

Runnning gst-plugins-good at edcc5be

Looks like this commit removed a bit too much
http://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit?id=b4aaa11f9733ba1800f7e88e3de911404aee7b11

Attached is a patch that unrefs the leaked caps.
Comment 1 Tim-Philipp Müller 2015-04-23 14:12:12 UTC
Pushed, thanks for the patch!

commit 3528046773c3c179751c1e727f9a13369d5578ed
Author: Jesper Larsen <knorr.jesper@gmail.com>
Date:   Thu Apr 23 09:55:59 2015 +0200

    rtspsrc: Fix RTCP caps leak
    
    https://bugzilla.gnome.org//show_bug.cgi?id=748353