[gstreamer-bugs] [Bug 341873] New: [alsasink] minor memory leak, uses unprotected static variable

GStreamer (bugzilla.gnome.org) bugzilla-daemon at bugzilla.gnome.org
Mon May 15 08:57:26 PDT 2006


Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=341873
 GStreamer | gst-plugins-base | Ver: HEAD CVS

           Summary: [alsasink] minor memory leak, uses unprotected static
                    variable
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: minor
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: t.i.m at zen.co.uk
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


alsasink uses a

  static snd_output_t  *output;

variable unprotected. I don't think thread-safety is actually an issue here,
but the allocated snd_output_t structure may leak if there is more than one
alsasink instance at a time, because the second instance will just overwrite
the previously allocated snd_output_t.

Patch attached. Might be a bit overdone, maybe we can just remove this stuff
altogether?


-- 
Configure bugmail: http://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.




More information about the Gstreamer-bugs mailing list