Am 22.02.2013 07:44, schrieb Vinson Lee:
Fixes out-of-bounds write reported by Coverity.

Signed-off-by: Vinson Lee <v...@freedesktop.org>

Reviewed-by: Christian König <christian.koe...@amd.com>

---
  src/gallium/auxiliary/vl/vl_winsys_dri.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri.c 
b/src/gallium/auxiliary/vl/vl_winsys_dri.c
index 560c914..59c02bc 100644
--- a/src/gallium/auxiliary/vl/vl_winsys_dri.c
+++ b/src/gallium/auxiliary/vl/vl_winsys_dri.c
@@ -338,7 +338,7 @@ vl_screen_create(Display *display, int screen)
        goto free_screen;
device_name_length = xcb_dri2_connect_device_name_length(connect);
-   device_name = CALLOC(1, device_name_length);
+   device_name = CALLOC(1, device_name_length + 1);
     memcpy(device_name, xcb_dri2_connect_device_name(connect), 
device_name_length);
     device_name[device_name_length] = 0;
     fd = open(device_name, O_RDWR);

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to