The function strerror() is insecure in a multi-thread environment.
This patch uses rte_strerror() to replace it.

Cc: sta...@dpdk.org

Signed-off-by: Dengdui Huang <huangdeng...@huawei.com>
Acked-by: Chengwen Feng <fengcheng...@huawei.com>
---
 drivers/net/dpaa/dpaa_ethdev.c  | 4 ++--
 drivers/net/dpaa/fmlib/fm_lib.c | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/net/dpaa/dpaa_ethdev.c b/drivers/net/dpaa/dpaa_ethdev.c
index ef4c06db6a..214627986f 100644
--- a/drivers/net/dpaa/dpaa_ethdev.c
+++ b/drivers/net/dpaa/dpaa_ethdev.c
@@ -1080,7 +1080,7 @@ int dpaa_eth_rx_queue_setup(struct rte_eth_dev *dev, 
uint16_t queue_idx,
                ret = qman_init_fq(rxq, flags, &opts);
                if (ret) {
                        DPAA_PMD_ERR("Channel/Q association failed. fqid 0x%x "
-                               "ret:%d(%s)", rxq->fqid, ret, strerror(ret));
+                               "ret:%d(%s)", rxq->fqid, ret, 
rte_strerror(ret));
                        return ret;
                }
                if (dpaa_svr_family == SVR_LS1043A_FAMILY) {
@@ -1210,7 +1210,7 @@ dpaa_eth_eventq_attach(const struct rte_eth_dev *dev,
        ret = qman_init_fq(rxq, flags, &opts);
        if (ret) {
                DPAA_PMD_ERR("Ev-Channel/Q association failed. fqid 0x%x "
-                               "ret:%d(%s)", rxq->fqid, ret, strerror(ret));
+                               "ret:%d(%s)", rxq->fqid, ret, 
rte_strerror(ret));
                return ret;
        }
 
diff --git a/drivers/net/dpaa/fmlib/fm_lib.c b/drivers/net/dpaa/fmlib/fm_lib.c
index 1d6816050c..d4382d7065 100644
--- a/drivers/net/dpaa/fmlib/fm_lib.c
+++ b/drivers/net/dpaa/fmlib/fm_lib.c
@@ -103,7 +103,7 @@ fm_get_api_version(t_handle h_fm, ioc_fm_api_version_t 
*p_version)
        ret = ioctl(p_dev->fd, FM_IOC_GET_API_VERSION, p_version);
        if (ret) {
                DPAA_PMD_ERR("cannot get API version, error %i (%s)\n",
-                            errno, strerror(errno));
+                            errno, rte_strerror(errno));
                RETURN_ERROR(MINOR, E_INVALID_OPERATION, NO_MSG);
        }
        _fml_dbg("Finishing.\n");
@@ -276,7 +276,7 @@ fm_pcd_kg_scheme_set(t_handle h_fm_pcd,
        ret = ioctl(p_pcd_dev->fd, FM_PCD_IOC_KG_SCHEME_SET, params);
        if (ret) {
                DPAA_PMD_ERR("  cannot set kg scheme, error %i (%s)\n",
-                            errno, strerror(errno));
+                            errno, rte_strerror(errno));
                return NULL;
        }
 
@@ -310,7 +310,7 @@ fm_pcd_kg_scheme_delete(t_handle h_scheme)
 
        if (ioctl(p_pcd_dev->fd, FM_PCD_IOC_KG_SCHEME_DELETE, &id)) {
                DPAA_PMD_WARN("cannot delete kg scheme, error %i (%s)\n",
-                             errno, strerror(errno));
+                             errno, rte_strerror(errno));
                RETURN_ERROR(MINOR, E_INVALID_OPERATION, NO_MSG);
        }
 
-- 
2.33.0

Reply via email to