[LTP] [PATCH 1/2] aio_cancel: Fix incorrect strerror() usage
Martin Doucha
mdoucha@suse.cz
Tue Apr 28 15:32:11 CEST 2026
Each call of strerror() invalidates the previous return value. Split
print calls to avoid calling strerror() twice in the argument list.
Signed-off-by: Martin Doucha <mdoucha@suse.cz>
---
.../conformance/interfaces/aio_cancel/5-1.c | 5 +++--
.../conformance/interfaces/aio_cancel/7-1.c | 5 +++--
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/5-1.c b/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/5-1.c
index dd5b0bbfb..04306417e 100644
--- a/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/5-1.c
+++ b/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/5-1.c
@@ -131,8 +131,9 @@ int test_main(int argc PTS_ATTRIBUTE_UNUSED, char **argv PTS_ATTRIBUTE_UNUSED)
}
if (ret != exp_ret) {
- printf(TNAME " Bad task #%d result: %s (expected %s)\n",
- i, strerror(ret), strerror(exp_ret));
+ printf(TNAME " Bad task #%d result: %s",
+ i, strerror(ret));
+ printf(" (expected %s)\n", strerror(exp_ret));
cleanup_aio(fds, aiocb, BUF_NB);
return PTS_FAIL;
}
diff --git a/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/7-1.c b/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/7-1.c
index 36ce8bb12..0fdc1bef8 100644
--- a/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/7-1.c
+++ b/testcases/open_posix_testsuite/conformance/interfaces/aio_cancel/7-1.c
@@ -118,8 +118,9 @@ int test_main(int argc PTS_ATTRIBUTE_UNUSED, char **argv PTS_ATTRIBUTE_UNUSED)
}
if (ret != exp_ret) {
- printf(TNAME " Bad task #%d result: %s (expected %s)\n",
- i, strerror(ret), strerror(exp_ret));
+ printf(TNAME " Bad task #%d result: %s",
+ i, strerror(ret));
+ printf(" (expected %s)\n", strerror(exp_ret));
cleanup_aio(fds, aiocb, BUF_NB);
return PTS_FAIL;
}
--
2.53.0
More information about the ltp
mailing list