In mremap_move_multiple_vmas_split(), if data is corrupted after mremap, the control jumps to cleanup without marking the test as failed. Set success to false before moving to cleanup. Fixes: 7d6597dfef11 ("tools/testing/selftests: explicitly test split multi VMA mremap move") Acked-by: David Hildenbrand (Arm) Signed-off-by: Sarthak Sharma --- tools/testing/selftests/mm/mremap_test.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests/mm/mremap_test.c b/tools/testing/selftests/mm/mremap_test.c index 111cc3414073..514d2003b88d 100644 --- a/tools/testing/selftests/mm/mremap_test.c +++ b/tools/testing/selftests/mm/mremap_test.c @@ -682,6 +682,7 @@ static void mremap_move_multiple_vmas_split(unsigned int pattern_seed, if (chr != buf[j]) { ksft_print_msg("page %d offset %d corrupted, expected %d got %d\n", i, j, chr, buf[j]); + success = false; goto out_unmap; } } -- 2.53.0