diff mbox series

[v3,12/42] bus/fslmc: use rte strerror

Message ID	20231114123552.398072-13-huangdengdui@huawei.com (mailing list archive)
State	Changes Requested, archived
Delegated to:	David Marchand
Headers	From: Dengdui Huang <huangdengdui@huawei.com> To: <dev@dpdk.org> CC: <ferruh.yigit@amd.com>, <stephen@networkplumber.org>, <lihuisong@huawei.com>, <fengchengwen@huawei.com>, <liuyonglong@huawei.com> Subject: [PATCH v3 12/42] bus/fslmc: use rte strerror Date: Tue, 14 Nov 2023 20:35:22 +0800 Message-ID: <20231114123552.398072-13-huangdengdui@huawei.com> In-Reply-To: <20231114123552.398072-1-huangdengdui@huawei.com> References: <20231114082539.1858594-44-huangdengdui@huawei.com> <20231114123552.398072-1-huangdengdui@huawei.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain Precedence: list Errors-To: dev-bounces@dpdk.org
Series	replace strerror \| [v3,00/42] replace strerror [v3,01/42] devtools: forbid use of strerror [v3,02/42] eal: use rte strerror [v3,03/42] eventdev: use rte strerror [v3,04/42] latency: use rte strerror [v3,05/42] node: use rte strerror [v3,06/42] pdump: use rte strerror [v3,07/42] power: use rte strerror [v3,08/42] vhost: use rte strerror [v3,09/42] bpf: use rte strerror [v3,10/42] bus/cdx: use rte strerror [v3,11/42] bus/dpaa: use rte strerror [v3,12/42] bus/fslmc: use rte strerror [v3,13/42] bus/pci: use rte strerror [v3,14/42] bus/vdev: use rte strerror [v3,15/42] bus/vmbus: use rte strerror [v3,16/42] common/cnxk: use rte strerror [v3,17/42] common/mlx5: use rte strerror [v3,18/42] crypto/caam_jr: use rte strerror [v3,19/42] dma/idxd: use rte strerror [v3,20/42] net/af_packet: use rte strerror [v3,21/42] net/bnxt: use rte strerror [v3,22/42] net/af_xdp: use rte strerror [v3,23/42] net/bonding: use rte strerror [v3,24/42] net/dpaa: use rte strerror [v3,25/42] net/dpaa2: use rte strerror [v3,26/42] net/enetfec: use rte strerror [v3,27/42] net/failsafe: use rte strerror [v3,28/42] net/i40e: use rte strerror [v3,29/42] net/ice: use rte strerror [v3,30/42] net/ixgbe: use rte strerror [v3,31/42] net/memif: use rte strerror [v3,32/42] net/mlx4: use rte strerror [v3,33/42] net/mlx5: use rte strerror [v3,34/42] net/qede: use rte strerror [v3,35/42] net/sfc: use rte strerror [v3,36/42] net/tap: use rte strerror [v3,37/42] net/vhost: use rte strerror [v3,38/42] net/virtio: use rte strerror [v3,39/42] raw/ifpga: use rte strerror [v3,40/42] vdpa/ifc: use rte strerror [v3,41/42] vdpa/mlx5: use rte strerror [v3,42/42] vdpa/sfc: use rte strerror

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Dengdui Huang Nov. 14, 2023, 12:35 p.m. UTC

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

Cc: stable@dpdk.org

Signed-off-by: Dengdui Huang <huangdengdui@huawei.com>
Acked-by: Chengwen Feng <fengchengwen@huawei.com>
---
 drivers/bus/fslmc/fslmc_vfio.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff mbox series

Patch

diff --git a/drivers/bus/fslmc/fslmc_vfio.c b/drivers/bus/fslmc/fslmc_vfio.c
index 5966776a85..1cbbcf9019 100644
--- a/drivers/bus/fslmc/fslmc_vfio.c
+++ b/drivers/bus/fslmc/fslmc_vfio.c
@@ -481,7 +481,7 @@  fslmc_vfio_setup_device(const char *sysfs_base, const char *dev_addr,
 	if (ret) {
 		DPAA2_BUS_ERR("  %s cannot get group status, "
 				"error %i (%s)\n", dev_addr,
-				errno, strerror(errno));
+				errno, rte_strerror(errno));
 		close(vfio_group_fd);
 		rte_vfio_clear_group(vfio_group_fd);
 		return -1;
@@ -504,7 +504,7 @@  fslmc_vfio_setup_device(const char *sysfs_base, const char *dev_addr,
 		if (ret) {
 			DPAA2_BUS_ERR("  %s cannot add VFIO group to container, "
 					"error %i (%s)\n", dev_addr,
-					errno, strerror(errno));
+					errno, rte_strerror(errno));
 			close(vfio_group_fd);
 			close(vfio_container_fd);
 			rte_vfio_clear_group(vfio_group_fd);
@@ -551,7 +551,7 @@  fslmc_vfio_setup_device(const char *sysfs_base, const char *dev_addr,
 	ret = ioctl(*vfio_dev_fd, VFIO_DEVICE_GET_INFO, device_info);
 	if (ret) {
 		DPAA2_BUS_ERR("  %s cannot get device info, error %i (%s)",
-				dev_addr, errno, strerror(errno));
+				dev_addr, errno, rte_strerror(errno));
 		close(*vfio_dev_fd);
 		close(vfio_group_fd);
 		close(vfio_container_fd);
@@ -617,7 +617,7 @@  int rte_dpaa2_intr_enable(struct rte_intr_handle *intr_handle, int index)
 	if (ret) {
 		DPAA2_BUS_ERR("Error:dpaa2 SET IRQs fd=%d, err = %d(%s)",
 			      rte_intr_fd_get(intr_handle), errno,
-			      strerror(errno));
+			      rte_strerror(errno));
 		return ret;
 	}
 
@@ -667,7 +667,7 @@  rte_dpaa2_vfio_setup_intr(struct rte_intr_handle *intr_handle,
 		ret = ioctl(vfio_dev_fd, VFIO_DEVICE_GET_IRQ_INFO, &irq_info);
 		if (ret < 0) {
 			DPAA2_BUS_ERR("Cannot get IRQ(%d) info, error %i (%s)",
-				      i, errno, strerror(errno));
+				      i, errno, rte_strerror(errno));
 			return -1;
 		}
 
@@ -682,7 +682,7 @@  rte_dpaa2_vfio_setup_intr(struct rte_intr_handle *intr_handle,
 		fd = eventfd(0, EFD_NONBLOCK | EFD_CLOEXEC);
 		if (fd < 0) {
 			DPAA2_BUS_ERR("Cannot set up eventfd, error %i (%s)",
-				      errno, strerror(errno));
+				      errno, rte_strerror(errno));
 			return -1;
 		}