[igt-dev] [PATCH i-g-t] igt/gem_shrink: Fix reversed logic when mmap fails

Ashutosh Dixit ashutosh.dixit at intel.com
Mon Apr 13 18:25:25 UTC 2020


Can't touch the memory if mmap returns MAP_FAILED.

v2: s/(char *)-1)/MAP_FAILED/ (Mike Ruhl)

Fixes: 9cff3be127("igt/gem_shrink: Reduce oom spam")
Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
---
 tests/i915/gem_shrink.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/i915/gem_shrink.c b/tests/i915/gem_shrink.c
index 11aa9efc1..e434613eb 100644
--- a/tests/i915/gem_shrink.c
+++ b/tests/i915/gem_shrink.c
@@ -274,7 +274,7 @@ static void leak(int fd, uint64_t alloc)
 	ptr = mmap(NULL, alloc, PROT_READ | PROT_WRITE,
 		   MAP_ANON | MAP_PRIVATE | MAP_POPULATE,
 		   -1, 0);
-	if (ptr != (char *)-1)
+	if (ptr == MAP_FAILED)
 		return;
 
 	while (alloc) {
-- 
2.26.0



More information about the igt-dev mailing list