[gstreamer-bugs] [Bug 640071] pluginloader: do not leak the description string of blacklisted plugin

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=640071
  GStreamer | gstreamer (core) | unspecified

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.33
            Summary|gstpluginloader: do not     |pluginloader: do not leak
                   |leak the description string |the description string of
                   |                            |blacklisted plugin

--- Comment #4 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2011-01-24 16:47:59 UTC ---
Thanks, pushed a version that just assigns the const strings, since slomo also
agreed that there's no need to intern those:


commit 6b84f4573c876fb59c153bf082f2e9bdc9875f88
Author: Vincent Penquerc'h <vincent.penquerch at collabora.co.uk>
Date:   Thu Jan 20 14:37:25 2011 +0000

    gstpluginloader: do not leak the description string

    The description string was changed to an inlined string a while back.
    (But: no need to intern the const strings here, we just use the interning
    to avoid allocating duplicates and make memory management easier,
    since the strings will be around for the life-time of the app anyway).

    https://bugzilla.gnome.org/show_bug.cgi?id=640071

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.