selftests/nolibc: mmap_munmap_good: fix up return value
The other tests use 1 as failure, mmap_munmap_good uses -1 as failure, let's fix up this. Signed-off-by: Zhangjin Wu <falcon@tinylab.org> Reviewed-by: Thomas Weißschuh <linux@weissschuh.net> Signed-off-by: Willy Tarreau <w@1wt.eu>
This commit is contained in:
parent
447e56023f
commit
e7d0129df6
1 changed files with 4 additions and 4 deletions
|
@ -726,7 +726,7 @@ int test_mmap_munmap(void)
|
||||||
|
|
||||||
page_size = getpagesize();
|
page_size = getpagesize();
|
||||||
if (page_size < 0)
|
if (page_size < 0)
|
||||||
return -1;
|
return 1;
|
||||||
|
|
||||||
/* find a right file to mmap, existed and accessible */
|
/* find a right file to mmap, existed and accessible */
|
||||||
for (i = 0; files[i] != NULL; i++) {
|
for (i = 0; files[i] != NULL; i++) {
|
||||||
|
@ -737,7 +737,7 @@ int test_mmap_munmap(void)
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
if (ret == -1)
|
if (ret == -1)
|
||||||
return ret;
|
return 1;
|
||||||
|
|
||||||
ret = stat(files[i], &stat_buf);
|
ret = stat(files[i], &stat_buf);
|
||||||
if (ret == -1)
|
if (ret == -1)
|
||||||
|
@ -757,7 +757,7 @@ int test_mmap_munmap(void)
|
||||||
|
|
||||||
mem = mmap(NULL, length + offset - pa_offset, PROT_READ, MAP_SHARED, fd, pa_offset);
|
mem = mmap(NULL, length + offset - pa_offset, PROT_READ, MAP_SHARED, fd, pa_offset);
|
||||||
if (mem == MAP_FAILED) {
|
if (mem == MAP_FAILED) {
|
||||||
ret = -1;
|
ret = 1;
|
||||||
goto end;
|
goto end;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -765,7 +765,7 @@ int test_mmap_munmap(void)
|
||||||
|
|
||||||
end:
|
end:
|
||||||
close(fd);
|
close(fd);
|
||||||
return ret;
|
return !!ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue