diff options
author | Vidya Srinivas <vidya.srinivas@intel.com> | 2021-05-28 10:02:47 +0530 |
---|---|---|
committer | Petri Latvala <petri.latvala@intel.com> | 2021-06-21 13:29:17 +0300 |
commit | 80d50562d579e01b9ee5f3dd79caec54f24ee33a (patch) | |
tree | 720c36262316d88e49db7f5d40bdc83b4a12022e /tests/drm_read.c | |
parent | 138a29e30277b1039e9934fca5c782dc1e7a9f99 (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.c | 3 |
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); |