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 724957 - vaapidecode leaks the pool provided by upstream when it does not provide our meta
vaapidecode leaks the pool provided by upstream when it does not provide our ...
Status: RESOLVED FIXED
Product: gstreamer-vaapi
Classification: Other
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gstreamer-vaapi maintainer(s)
gstreamer-vaapi maintainer(s)
Depends on:
Blocks: 720305
 
 
Reported: 2014-02-22 14:47 UTC by Matthew Waters (ystreet00)
Modified: 2014-06-18 12:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
decode: plug a memory leak (1.09 KB, patch)
2014-02-22 14:48 UTC, Matthew Waters (ystreet00)
none Details | Review

Description Matthew Waters (ystreet00) 2014-02-22 14:47:13 UTC
See summary and attached patch
Comment 1 Matthew Waters (ystreet00) 2014-02-22 14:48:05 UTC
Created attachment 269995 [details] [review]
decode: plug a memory leak
Comment 2 Gwenole Beauchesne 2014-06-18 12:05:23 UTC
commit 3fbef25e13e231b34e4d50823c86bb28240f1318
Author: Matthew Waters <ystreet00@gmail.com>
Date:   Sun Feb 23 01:43:39 2014 +1100

    vaapidecode: plug a memory leak.
    
    It can happen that there is a pool provided that does not advertise
    the vappivideometa. We should unref that pool before using our own.
    
    Discovered with vaapidecode ! {glimagesink,cluttersink}
    
    https://bugzilla.gnome.org/show_bug.cgi?id=724957
    
    [fixed compilation by adding the missing semi-colon]
    Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>