[dpdk-dev] [PATCH] eal: fix API to get error string
Ferruh Yigit
ferruh.yigit at intel.com
Wed Oct 31 18:19:28 CET 2018
rte_strerror uses strerror_r(), and strerror_r() has two version of it.
- XSI-compliant version, (_POSIX_C_SOURCE >= 200112L) && ! _GNU_SOURCE
- GNU-specific version
Those two has different return types, so the exiting return type check
is not correct for GNU-specific version.
And this is causing failure in errno_autotest unit test.
Adding different implementation for FreeBSD and Linux.
Fixes: 016c32bd3e3d ("eal: cleanup strerror function")
Cc: stable at dpdk.org
Signed-off-by: Ferruh Yigit <ferruh.yigit at intel.com>
---
lib/librte_eal/common/eal_common_errno.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/lib/librte_eal/common/eal_common_errno.c b/lib/librte_eal/common/eal_common_errno.c
index 56b492f5f..fbbc71b0b 100644
--- a/lib/librte_eal/common/eal_common_errno.c
+++ b/lib/librte_eal/common/eal_common_errno.c
@@ -38,9 +38,17 @@ rte_strerror(int errnum)
case E_RTE_NO_CONFIG:
return "Missing rte_config structure";
default:
+#ifdef RTE_EXEC_ENV_BSDAPP
if (strerror_r(errnum, ret, RETVAL_SZ) != 0)
snprintf(ret, RETVAL_SZ, "Unknown error%s %d",
sep, errnum);
+#else
+ /*
+ * _GNU_SOURCE version, error string is not always
+ * strored in "ret" buffer, need to use return value
+ */
+ ret = strerror_r(errnum, ret, RETVAL_SZ);
+#endif
}
return ret;
--
2.17.2
More information about the dev
mailing list