summaryrefslogtreecommitdiff
path: root/tests/drm_read.c
diff options
context:
space:
mode:
authorVidya Srinivas <vidya.srinivas@intel.com>2021-05-28 10:02:47 +0530
committerPetri Latvala <petri.latvala@intel.com>2021-06-21 13:29:17 +0300
commit80d50562d579e01b9ee5f3dd79caec54f24ee33a (patch)
tree720c36262316d88e49db7f5d40bdc83b4a12022e /tests/drm_read.c
parent138a29e30277b1039e9934fca5c782dc1e7a9f99 (diff)
tests/drm_read: Fix subtest invalid-buffer
Using (void *)-1 directly in read is aborting on chrome systems. Following message is seen. Starting subtest: invalid-buffer *** buffer overflow detected ***: terminated Received signal SIGABRT. Stack trace: Aborted (core dumped) Patch just adds a pointer variable and uses it in read. Signed-off-by: Vidya Srinivas <vidya.srinivas@intel.com> Tested-by: Mark Yacoub <markyacoub@chromium.org> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'tests/drm_read.c')
-rw-r--r--tests/drm_read.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/drm_read.c b/tests/drm_read.c
index ccf9d822..2fdec5be 100644
--- a/tests/drm_read.c
+++ b/tests/drm_read.c
@@ -103,10 +103,11 @@ static void teardown(int fd)
static void test_invalid_buffer(int in)
{
int fd = setup(in, 0);
+ void *add = (void *)-1;
alarm(1);
- igt_assert_eq(read(fd, (void *)-1, 4096), -1);
+ igt_assert_eq(read(fd, add, 4096), -1);
igt_assert_eq(errno, EFAULT);
teardown(fd);