[Intel-gfx] [PATCH 2/7] display: prevent a NULL pointer dereference in intel_set_scanout_pixmap

Martin Peres martin.peres at linux.intel.com
Fri Feb 12 16:31:24 UTC 2016


Caught by Klockwork.

Signed-off-by: Martin Peres <martin.peres at linux.intel.com>
---
 src/uxa/intel_display.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/uxa/intel_display.c b/src/uxa/intel_display.c
index 8bf0184..44744a5 100644
--- a/src/uxa/intel_display.c
+++ b/src/uxa/intel_display.c
@@ -688,6 +688,11 @@ intel_set_scanout_pixmap(xf86CrtcPtr crtc, PixmapPtr ppix)
 	}
 
 	bo = intel_get_pixmap_bo(ppix);
+	if (!bo) {
+		ErrorF("pixmap is not backed by a BO\n");
+		return FALSE;
+	}
+
 	if (intel->front_buffer) {
 		ErrorF("have front buffer\n");
 	}
-- 
2.7.1



More information about the Intel-gfx mailing list