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 781695 - vaapi: leak of vpp surface in GstVaapiWindowWayland
vaapi: leak of vpp surface in GstVaapiWindowWayland
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer-vaapi
git master
Other Linux
: Normal normal
: 1.11.91
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-04-25 07:29 UTC by Hyunjun Ko
Modified: 2017-04-25 09:36 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
libs: windows: wayland: fix leak if failure of sync (1.25 KB, patch)
2017-04-25 07:30 UTC, Hyunjun Ko 		committed Details | Review

Description Hyunjun Ko 2017-04-25 07:29:24 UTC 
See commit message.
Comment 1 Hyunjun Ko 2017-04-25 07:30:28 UTC 
Created attachment 350371 [details] [review]
libs: windows: wayland: fix leak if failure of sync

Sometimes gst_vaapi_window_wayland_sync returns FALSE when poll returns EBUSY
during destruction.
In this case, if GstVaapiWindow is using vpp, leak of vpp surface happens.
This surface is not attached to anything at this moment, so we should release
it manually.
Comment 2 Víctor Manuel Jáquez Leal 2017-04-25 09:18:21 UTC 
Comment on attachment 350371 [details] [review]
libs: windows: wayland: fix leak if failure of sync

lgtm
Comment 3 Víctor Manuel Jáquez Leal 2017-04-25 09:35:47 UTC 
Attachment 350371 [details] pushed as 2d7c763 - libs: windows: wayland: fix leak if failure of sync