From 99f9b077c62e14ba955b9c1f7afda47f7799d317 Mon Sep 17 00:00:00 2001 From: Daniel Stone Date: Thu, 5 Apr 2018 16:47:39 +0100 Subject: [PATCH] modesetting: Actually get framebuffer ID We would fail to get the FB ID if it wasn't already imported, since we were checking to see if the pointer was NULL (it never was) rather than if the content of the pointer was 0. Signed-off-by: Daniel Stone Reported-by: Olivier Fourdan Tested-by: Olivier Fourdan Reviewed-by: Olivier Fourdan --- hw/xfree86/drivers/modesetting/drmmode_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/xfree86/drivers/modesetting/drmmode_display.c b/hw/xfree86/drivers/modesetting/drmmode_display.c index cdcd4f3f3..322ef050b 100644 --- a/hw/xfree86/drivers/modesetting/drmmode_display.c +++ b/hw/xfree86/drivers/modesetting/drmmode_display.c @@ -624,7 +624,7 @@ drmmode_crtc_get_fb_id(xf86CrtcPtr crtc, uint32_t *fb_id, int *x, int *y) *y = crtc->y; } - if (fb_id == 0) { + if (*fb_id == 0) { ret = drmmode_bo_import(drmmode, &drmmode->front_bo, &drmmode->fb_id); if (ret < 0) {