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 766561 - gst-plugins-good: fix leaks in various tests
gst-plugins-good: fix leaks in various tests
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal minor
: 1.9.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks: 767856 772644
 
 
Reported: 2016-05-17 10:47 UTC by Guillaume Desmottes
Modified: 2016-10-09 13:54 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
deinterleave: fix leaks (1.39 KB, patch)
2016-05-17 10:48 UTC, Guillaume Desmottes
committed Details | Review
videomixer: fix event leaks in test (1.17 KB, patch)
2016-05-17 10:48 UTC, Guillaume Desmottes
none Details | Review
videomixer: fix event leaks in test (1.17 KB, patch)
2016-05-17 10:50 UTC, Guillaume Desmottes
committed Details | Review
interleave: fix message leaks in test (1.08 KB, patch)
2016-05-17 10:50 UTC, Guillaume Desmottes
committed Details | Review
fix buffer leaks in tests (4.56 KB, patch)
2016-05-17 10:52 UTC, Guillaume Desmottes
committed Details | Review

Description Guillaume Desmottes 2016-05-17 10:47:45 UTC
.
Comment 1 Guillaume Desmottes 2016-05-17 10:48:26 UTC
Created attachment 328056 [details] [review]
deinterleave: fix leaks

- Flush the bus so messages aren't leaked
- Fix pad leak
Comment 2 Guillaume Desmottes 2016-05-17 10:48:49 UTC
Created attachment 328057 [details] [review]
videomixer: fix event leaks in test
Comment 3 Guillaume Desmottes 2016-05-17 10:50:21 UTC
Created attachment 328058 [details] [review]
videomixer: fix event leaks in test
Comment 4 Guillaume Desmottes 2016-05-17 10:50:53 UTC
Created attachment 328059 [details] [review]
interleave: fix message leaks in test

Flush the bus when cleaning up so pending messages are destroyed.
Comment 5 Guillaume Desmottes 2016-05-17 10:52:48 UTC
Created attachment 328060 [details] [review]
fix buffer leaks in tests

Need to call gst_check_drop_buffers() to release the buffers exchanged
during the test.
Comment 6 Sebastian Dröge (slomo) 2016-06-21 07:51:54 UTC
Attachment 328056 [details] pushed as c7621e2 - deinterleave: fix leaks
Attachment 328058 [details] pushed as 23c8f71 - videomixer: fix event leaks in test
Attachment 328059 [details] pushed as fb41b30 - interleave: fix message leaks in test
Attachment 328060 [details] pushed as 6ae9c23 - fix buffer leaks in tests