diff options
| author | Thomas Weißschuh <linux@weissschuh.net> | 2025-08-21 17:13:02 +0200 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2025-09-08 15:54:39 +0100 |
| commit | 740cdafd0d998903c1faeee921028a8a78698be5 (patch) | |
| tree | df9eda3152a967f1507101da738159bc25198dab | |
| parent | 50af02425afc72b1b47c4a0a0b9c9bdaa1a1b347 (diff) | |
kselftest/arm64/gcs: Correctly check return value when disabling GCS
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>
| -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); |