diff options
| author | Thomas Weißschuh <linux@weissschuh.net> | 2025-08-21 17:13:02 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-10-15 12:03:14 +0200 |
| commit | 2714a1ead2448616e2126e94974bb39574655b3c (patch) | |
| tree | 27cef7ae66e6727f12e0ddd577d76379c9a8c548 /tools | |
| parent | 8262b855cb717ab39218b532c55ad9b77f294fd8 (diff) | |
| download | linux-2714a1ead2448616e2126e94974bb39574655b3c.tar.gz linux-2714a1ead2448616e2126e94974bb39574655b3c.tar.bz2 linux-2714a1ead2448616e2126e94974bb39574655b3c.zip | |
kselftest/arm64/gcs: Correctly check return value when disabling GCS
[ Upstream commit 740cdafd0d998903c1faeee921028a8a78698be5 ]
The return value was not assigned to 'ret', so the check afterwards
does not do anything.
Fixes: 3d37d4307e0f ("kselftest/arm64: Add very basic GCS test program")
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Reviewed-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Will Deacon <will@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/testing/selftests/arm64/gcs/basic-gcs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/arm64/gcs/basic-gcs.c b/tools/testing/selftests/arm64/gcs/basic-gcs.c index 54f9c888249d..100d2a983155 100644 --- a/tools/testing/selftests/arm64/gcs/basic-gcs.c +++ b/tools/testing/selftests/arm64/gcs/basic-gcs.c @@ -410,7 +410,7 @@ int main(void) } /* One last test: disable GCS, we can do this one time */ - my_syscall5(__NR_prctl, PR_SET_SHADOW_STACK_STATUS, 0, 0, 0, 0); + ret = my_syscall5(__NR_prctl, PR_SET_SHADOW_STACK_STATUS, 0, 0, 0, 0); if (ret != 0) ksft_print_msg("Failed to disable GCS: %d\n", ret); |
