diff mbox series

[v5,05/52] latency: replace strerror with reentrant version

Message ID	20241104111037.3632161-6-huangdengdui@huawei.com (mailing list archive)
State	Superseded, archived
Delegated to:	Thomas Monjalon
Headers	From: Dengdui Huang <huangdengdui@huawei.com> To: <dev@dpdk.org> CC: <ajit.khaparde@broadcom.com>, <thomas@monjalon.net>, <ferruh.yigit@amd.com>, <stephen@networkplumber.org>, <mb@smartsharesystems.com>, <david.marchand@redhat.com>, <iboukris@gmail.com>, <lihuisong@huawei.com>, <fengchengwen@huawei.com>, <haijie1@huawei.com>, <liuyonglong@huawei.com> Subject: [PATCH v5 05/52] latency: replace strerror with reentrant version Date: Mon, 4 Nov 2024 19:09:50 +0800 Message-ID: <20241104111037.3632161-6-huangdengdui@huawei.com> In-Reply-To: <20241104111037.3632161-1-huangdengdui@huawei.com> References: <20231114082539.1858594-1-huangdengdui@huawei.com> <20241104111037.3632161-1-huangdengdui@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit Precedence: list Errors-To: dev-bounces@dpdk.org
Series	replace strerror \| [v5,00/52] replace strerror [v5,01/52] eal: fix use rte errno for libc function [v5,02/52] devtools: forbid use of strerror [v5,03/52] eal: replace strerror with reentrant version [v5,04/52] eventdev: replace strerror with reentrant version [v5,05/52] latency: replace strerror with reentrant version [v5,06/52] node: replace strerror with reentrant version [v5,07/52] pdump: replace strerror with reentrant version [v5,08/52] pdump: use rte strerror [v5,09/52] power: replace strerror with reentrant version [v5,10/52] vhost: replace strerror with reentrant version [v5,11/52] bpf: replace strerror with reentrant version [v5,12/52] bus/cdx: replace strerror with reentrant version [v5,13/52] bus/dpaa: replace strerror with reentrant version [v5,14/52] bus/fslmc: replace strerror with reentrant version [v5,15/52] bus/pci: replace strerror with reentrant version [v5,16/52] bus/vmbus: replace strerror with reentrant version [v5,17/52] bus/vdev: use rte strerror [v5,18/52] common/cnxk: replace strerror with reentrant version [v5,19/52] common/mlx5: replace strerror with reentrant version [v5,20/52] common/mlx5: use rte strerror [v5,21/52] crypto/caam_jr: replace strerror with reentrant version [v5,22/52] dma/idxd: replace strerror with reentrant version [v5,23/52] net/bnxt: replace strerror with reentrant version [v5,24/52] net/af_xdp: replace strerror with reentrant version [v5,25/52] net/af_xdp: use rte strerror [v5,26/52] net/bonding: replace strerror with reentrant version [v5,27/52] net/bonding: use rte strerror [v5,28/52] net/cnxk: replace strerror with reentrant version [v5,29/52] net/dpaa: replace strerror with reentrant version [v5,30/52] net/dpaa2: replace strerror with reentrant version [v5,31/52] net/enetfec: replace strerror with reentrant version [v5,32/52] net/failsafe: replace strerror with reentrant version [v5,33/52] net/failsafe: use rte strerror [v5,34/52] net/i40e: replace strerror with reentrant version [v5,35/52] net/ice: replace strerror with reentrant version [v5,36/52] net/ixgbe: replace strerror with reentrant version [v5,37/52] net/memif: replace strerror with reentrant version [v5,38/52] net/memif: use rte strerror [v5,39/52] net/mlx4: replace strerror with reentrant version [v5,40/52] net/mlx4: use rte strerror [v5,41/52] net/mlx5: replace strerror with reentrant version [v5,42/52] net/mlx5: use rte strerror [v5,43/52] net/qede: replace strerror with reentrant version [v5,44/52] net/sfc: replace strerror with reentrant version [v5,45/52] net/tap: replace strerror with reentrant version [v5,46/52] net/tap: use rte strerror [v5,47/52] net/vhost: replace strerror with reentrant version [v5,48/52] net/virtio: replace strerror with reentrant version [v5,49/52] raw/ifpga: replace strerror with reentrant version [v5,50/52] vdpa/ifc: replace strerror with reentrant version [v5,51/52] vdpa/mlx5: replace strerror with reentrant version [v5,52/52] vdpa/sfc: replace strerror with reentrant version

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

huangdengdui Nov. 4, 2024, 11:09 a.m. UTC

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

Signed-off-by: Dengdui Huang <huangdengdui@huawei.com>
Acked-by: Chengwen Feng <fengchengwen@huawei.com>
Acked-by: Morten Brørup <mb@smartsharesystems.com>
Acked-by: Huisong Li <lihuisong@huawei.com>
---
 lib/latencystats/rte_latencystats.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff mbox series

Patch

diff --git a/lib/latencystats/rte_latencystats.c b/lib/latencystats/rte_latencystats.c
index 6a261309f9..8957f6f6ac 100644
--- a/lib/latencystats/rte_latencystats.c
+++ b/lib/latencystats/rte_latencystats.c
@@ -220,6 +220,7 @@  rte_latencystats_init(uint64_t app_samp_intvl,
 	struct rxtx_cbs *cbs = NULL;
 	const char *ptr_strings[NUM_LATENCY_STATS] = {0};
 	const struct rte_memzone *mz = NULL;
+	char errmsg[RTE_STRERR_BUFSIZE];
 	const unsigned int flags = 0;
 	int ret;
 
@@ -266,9 +267,11 @@  rte_latencystats_init(uint64_t app_samp_intvl,
 
 		ret = rte_eth_dev_info_get(pid, &dev_info);
 		if (ret != 0) {
+			if (strerror_r(-ret, errmsg, sizeof(errmsg)) != 0)
+				snprintf(errmsg, sizeof(errmsg), "Unknown error %d", -ret);
 			LATENCY_STATS_LOG(INFO,
 				"Error during getting device (port %u) info: %s",
-				pid, strerror(-ret));
+				pid, errmsg);
 
 			continue;
 		}
@@ -303,6 +306,7 @@  rte_latencystats_uninit(void)
 	int ret = 0;
 	struct rxtx_cbs *cbs = NULL;
 	const struct rte_memzone *mz = NULL;
+	char errmsg[RTE_STRERR_BUFSIZE];
 
 	/** De register Rx/Tx callbacks */
 	RTE_ETH_FOREACH_DEV(pid) {
@@ -310,9 +314,11 @@  rte_latencystats_uninit(void)
 
 		ret = rte_eth_dev_info_get(pid, &dev_info);
 		if (ret != 0) {
+			if (strerror_r(-ret, errmsg, sizeof(errmsg)) != 0)
+				snprintf(errmsg, sizeof(errmsg), "Unknown error %d", -ret);
 			LATENCY_STATS_LOG(INFO,
 				"Error during getting device (port %u) info: %s",
-				pid, strerror(-ret));
+				pid, errmsg);
 
 			continue;
 		}