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 595529 - Screenshot cause memory leakage.
Screenshot cause memory leakage.
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: GStreamer backend
2.27.x
Other All
: Normal major
: ---
Assigned To: Maintainer alias for GStreamer component of Totem
Maintainer alias for GStreamer component of Totem
Depends on:
Blocks:
 
 
Reported: 2009-09-18 02:56 UTC by HU Kan
Modified: 2009-09-21 16:14 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch for screenshot memory leak (467 bytes, patch)
2009-09-21 01:45 UTC, HU Kan 		none Details | Review
Bug 595529 - Screenshot cause memory leakage (2.40 KB, patch)
2009-09-21 15:16 UTC, Bastien Nocera 		none Details | Review

Description HU Kan 2009-09-18 02:56:13 UTC 
for screen shot function, buffer get from video-sink element of playbin did not unreffed after bvw_frame_conv_convert, should add unref in feed_fakesrc function of gstscreenshot.c
Comment 1 Philip Withnall 2009-09-18 12:32:25 UTC 
I'm not quite sure what you mean here. Could you provide a patch fixing the issue?
Comment 2 HU Kan 2009-09-21 01:45:09 UTC 
Created attachment 143560 [details] [review]
patch for screenshot memory leak
Comment 3 Bastien Nocera 2009-09-21 15:16:58 UTC 
Created attachment 143611 [details] [review]
Bug 595529 - Screenshot cause memory leakage

We take ownership of the buffer, but never actually unref it,
do this, even on error. With help from Tim.