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 484990 - memleak - missing g_free() rtpmanager/gstrtpbin.c
memleak - missing g_free() rtpmanager/gstrtpbin.c
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 0.10.6
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-10-09 09:29 UTC by Laurent Glayal
Modified: 2007-10-09 10:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Laurent Glayal 2007-10-09 09:29:45 UTC 
Missing g_free for string at gstrtpbin.c:911 .

Patch :
Index: gstrtpbin.c
===================================================================
RCS file: /cvs/gstreamer/gst-plugins-bad/gst/rtpmanager/gstrtpbin.c,v
retrieving revision 1.30
diff -u -r1.30 gstrtpbin.c
--- gstrtpbin.c 8 Oct 2007 10:39:34 -0000       1.30
+++ gstrtpbin.c 9 Oct 2007 09:25:33 -0000
@@ -912,6 +912,7 @@
   templ = gst_static_pad_template_get (&rtpbin_sync_sink_template);
   stream->sync_pad = gst_pad_new_from_template (templ, padname);
   gst_object_unref (templ);
+  g_free(padname);
   gst_object_ref (stream->sync_pad);
   gst_object_sink (stream->sync_pad);
   gst_pad_set_element_private (stream->sync_pad, stream);
Comment 1 Wim Taymans 2007-10-09 10:01:55 UTC 
        Patch by: Laurent Glayal <spglegle at yahoo dot fr>

        * gst/rtpmanager/gstrtpbin.c: (create_stream),
        (gst_rtp_bin_class_init):
        Fix memleak. Fixes #484990.