From 225a0ee18abd9f3a67e0b81e1b1c103fe82a97a8 Mon Sep 17 00:00:00 2001 From: Chris Michael Date: Wed, 1 May 2013 21:26:02 -0400 Subject: [PATCH] Fix not checking return value of drmIoctl function call to map dumb buffer in drm_fb_create_dumb, the return value of the drmIoctl function call to map the dumb buffer was never checked, thus the following "if (ret)" check was invalid as it was checking the previous return value from the above drmModeAddFB call. Signed-off-by: Chris Michael --- src/compositor-drm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index da1ba79e8..890eba7fd 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -255,8 +255,7 @@ drm_fb_create_dumb(struct drm_compositor *ec, unsigned width, unsigned height) memset(&map_arg, 0, sizeof(map_arg)); map_arg.handle = fb->handle; - drmIoctl(fb->fd, DRM_IOCTL_MODE_MAP_DUMB, &map_arg); - + ret = drmIoctl(fb->fd, DRM_IOCTL_MODE_MAP_DUMB, &map_arg); if (ret) goto err_add_fb;