summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Olbrich <m.olbrich@pengutronix.de>2020-07-14 16:14:50 +0200
committerSimon Ser <contact@emersion.fr>2020-08-13 18:50:57 +0000
commitad41ad968afbab4c56cb81becf79bb47d575d388 (patch)
treef2ce1c307bc2d2d5ead8a87db3f7c80996f1f9ef
parentc5ea495f7adec6137c23e094b498816569b0d20f (diff)
downloadweston-ad41ad968afbab4c56cb81becf79bb47d575d388.tar.gz
gl-renderer: remove incorrect assertion
The refcount is not zero if the corresponding buffer is attached to multiple surfaces. Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
-rw-r--r--libweston/renderer-gl/gl-renderer.c7
1 files changed, 2 insertions, 5 deletions
diff --git a/libweston/renderer-gl/gl-renderer.c b/libweston/renderer-gl/gl-renderer.c
index 937ebe86..d5bfe830 100644
--- a/libweston/renderer-gl/gl-renderer.c
+++ b/libweston/renderer-gl/gl-renderer.c
@@ -2534,7 +2534,6 @@ gl_renderer_attach_dmabuf(struct weston_surface *surface,
struct gl_surface_state *gs = get_surface_state(surface);
struct dmabuf_image *image;
int i;
- int ret;
if (!gr->has_dmabuf_import) {
linux_dmabuf_buffer_send_server_error(dmabuf,
@@ -2580,10 +2579,8 @@ gl_renderer_attach_dmabuf(struct weston_surface *surface,
/* The dmabuf_image should have been created during the import */
assert(image != NULL);
- for (i = 0; i < image->num_images; ++i) {
- ret = egl_image_unref(image->images[i]);
- assert(ret == 0);
- }
+ for (i = 0; i < image->num_images; ++i)
+ egl_image_unref(image->images[i]);
if (!import_known_dmabuf(gr, image)) {
linux_dmabuf_buffer_send_server_error(dmabuf, "EGL dmabuf import failed");