commit 07684193931e98d55234e80861325f4e7e587449
author Pekka Paalanen <ppaalanen@gmail.com> Tue Jul 31 13:21:12 2012 +0300
committer Kristian Høgsberg <krh@bitplanet.net> Tue Jul 31 22:27:23 2012 -0400
tree 2057413d606e3f33f2a86f4e9e45699a38d692ef
parent 55b7cb24e0acb77e674f886e4557519e7177e359

simple-touch: fix off-by-one in position checks

Fix the off by one error in checking whether we can draw the marker
without exceeding buffer dimensions.

Fixes a segfault.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

clients/simple-touch.c

diff --git a/clients/simple-touch.c b/clients/simple-touch.c
index 3a1d940..f8afbb0 100644
--- a/clients/simple-touch.c
+++ b/clients/simple-touch.c
@@ -118,8 +118,8 @@
 	else
 		c = 0xffffffff;
 
-	if (x < 2 || touch->width - 2 < x ||
-	    y < 2 || touch->height - 2 < y)
+	if (x < 2 || x >= touch->width - 2 ||
+	    y < 2 || y >= touch->height - 2)
 		return;
 
 	p = (uint32_t *) touch->data + (x - 2) + (y - 2) * touch->width;