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 674011 - videopool: fix mem leak
videopool: fix mem leak
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.11.x
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-04-12 19:24 UTC by Víctor Manuel Jáquez Leal
Modified: 2012-04-14 08:36 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
videopool: fix mem leak (1.03 KB, patch)
2012-04-12 19:25 UTC, Víctor Manuel Jáquez Leal 		none Details | Review

Description Víctor Manuel Jáquez Leal 2012-04-12 19:24:00 UTC 
When setting its config, the pool increase the ref count of the allocator, but at finalize the ref count is also increased rather than decreased.
    
This one-liner patch changes the gst_allocator_ref() for gst_allocator_unref()
Comment 1 Víctor Manuel Jáquez Leal 2012-04-12 19:25:16 UTC 
Created attachment 211953 [details] [review]
videopool: fix mem leak
Comment 2 Wim Taymans 2012-04-14 08:36:44 UTC 
commit 3f3dceb6751d3047388f78ac6badd8168db7bac1
Author: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
Date:   Thu Apr 12 21:13:08 2012 +0200

    videopool: fix mem leak
    
    When setting its config, the pool increase the ref count of the allocator, but
    at finalize the ref count is also increased rather than decreased.
    
    This one-liner patch changes the gst_allocator_ref() for gst_allocator_unref()
    
    Fixes https://bugzilla.gnome.org/show_bug.cgi?id=674011