The DDX clearly is making Klockwork unhappy but I found those potential issues. Let's use them as a basis for discussion on the real code that should be written. Martin Peres (7): device: prevent a NULL pointer dereference in __intel_peek_fd display: prevent a NULL pointer dereference in intel_set_scanout_pixmap sna: check for NULL before dereferencing a pointer sna: assert a pointer is non-NULL before dereferencing it sna/cursor: add an assert on cursor->image sna_video_sprite: add asserts to catch invalid pipe# sna: add an assert in the lengthy sna_drawable_use_bo src/intel_device.c | 5 ++++- src/sna/sna_accel.c | 5 ++++- src/sna/sna_display.c | 3 ++- src/sna/sna_video_sprite.c | 3 +++ src/uxa/intel_display.c | 5 +++++ 5 files changed, 18 insertions(+), 3 deletions(-) -- 2.7.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx