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 408278 - [videorate] memory leak
[videorate] memory leak
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
0.10.11
Other Linux
: Normal minor
: 0.10.12
Assigned To: Tim-Philipp Müller
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-02-15 15:38 UTC by Yves Lefebvre
Modified: 2007-02-16 10:15 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch for this bug (396 bytes, patch)
2007-02-15 15:50 UTC, Yves Lefebvre 		committed Details | Review

Description Yves Lefebvre 2007-02-15 15:38:33 UTC 
I found a memory leak of caps with gstvideorate with this pipeline

gst-launch-0.10  -v videotestsrc num-buffers=10 ! videorate !  fakesink

will add the patch soon
Comment 1 Yves Lefebvre 2007-02-15 15:50:00 UTC 
Created attachment 82611 [details] [review]
patch for this bug

before fix (valgrind output) : 
==20910== LEAK SUMMARY:
==20910==    definitely lost: 36 bytes in 1 blocks.
==20910==    indirectly lost: 120 bytes in 10 blocks.
==20910==      possibly lost: 0 bytes in 0 blocks.
==20910==    still reachable: 364,398 bytes in 5,237 blocks.

after fix (less reachable memory):
==8391== LEAK SUMMARY:
==8391==    definitely lost: 36 bytes in 1 blocks.
==8391==    indirectly lost: 120 bytes in 10 blocks.
==8391==      possibly lost: 0 bytes in 0 blocks.
==8391==    still reachable: 155,636 bytes in 2,279 blocks.
Comment 2 Tim-Philipp Müller 2007-02-16 10:15:17 UTC 
Can't make this show up in valgrind for me, but the patch is obviously correct. Applied to CVS, thanks!

 2007-02-16  Tim-Philipp Müller  <tim at centricular dot net>

       Patch by: Yves Lefebvre  <ivanohe abacom com>

       * gst/videorate/gstvideorate.c: (gst_video_rate_setcaps):
         Don't leak caps. Fixes #408278.