[Bug 785065] encodebin: fix possible pad ref leak in error code path

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Jul 19 08:10:28 UTC 2017


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

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|git master                  |1.13.1
            Summary|encoding:Possible memory    |encodebin: fix possible pad
                   |leak in gstencodebin.c      |ref leak in error code path
           Severity|critical                    |minor

--- Comment #3 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Thanks for the patch, but it's not quite complete.

There's a compiler warning about how srcpad might be used uninitialized, and we
should also reset srcpad to NULL when we do properly unref it in the code, so
that we don't accidentally unref it twice when we hit an error.

I've fixed it up a bit and pushed it:

commit 3cc824c9cef76c93c3318f3691f8a51ba7c8f9c3
Author: Satya Prakash Gupta <sp.gupta at samsung.com>
Date:   Wed Jul 19 09:02:26 2017 +0530

    encodebin: fix possible pad ref leak in error code path

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

-- 
You are receiving this mail because:
You are the QA Contact for the bug.


More information about the gstreamer-bugs mailing list