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 361456 - [basertppayload] Memory leak
[basertppayload] Memory leak
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal minor
: 0.10.11
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-10-11 15:12 UTC by Sebastien Cote
Modified: 2006-10-12 19:09 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
fix (802 bytes, patch)
2006-10-11 15:13 UTC, Sebastien Cote 		committed Details | Review

Description Sebastien Cote 2006-10-11 15:12:44 UTC 
The base rtp payloader object doesn't free the media and encoding_name strings upon destruction which result in a memory leak.
Comment 1 Sebastien Cote 2006-10-11 15:13:39 UTC 
Created attachment 74491 [details] [review]
fix

Initialize the strings in _init and free them _finalize.
Comment 2 Tim-Philipp Müller 2006-10-12 19:09:15 UTC 
Shows that we don't even have the most basic unit tests for the base payloader, or any payloader really ... *sigh*

Thanks for the patch! Committed to CVS:

 2006-10-12  Tim-Philipp Müller  <tim at centricular dot net>

        Patch by: Sebastien Cote  <sebas642 at yahoo.ca>

        * gst-libs/gst/rtp/gstbasertppayload.c: (gst_basertppayload_init),
        (gst_basertppayload_finalize):
          Fix two small memory leaks (#361456).