diff mbox series

[04/16] bus/vdev: verify strdup return value

Message ID	20231110063653.5948-5-fengchengwen@huawei.com (mailing list archive)
State	Superseded, archived
Delegated to:	Thomas Monjalon
Headers	From: Chengwen Feng <fengchengwen@huawei.com> To: <thomas@monjalon.net>, <ferruh.yigit@amd.com>, Gaetan Rivet <grive@u256.net>, Ray Kinsella <mdr@ashroe.eu>, Xueming Li <xuemingl@nvidia.com> CC: <dev@dpdk.org> Subject: [PATCH 04/16] bus/vdev: verify strdup return value Date: Fri, 10 Nov 2023 06:36:41 +0000 Message-ID: <20231110063653.5948-5-fengchengwen@huawei.com> In-Reply-To: <20231110063653.5948-1-fengchengwen@huawei.com> References: <20231110063653.5948-1-fengchengwen@huawei.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: list Errors-To: dev-bounces@dpdk.org
Series	verify strdup return value \| [00/16] verify strdup return value [01/16] eal: verify strdup return value [02/16] bus/dpaa: verify strdup return value [03/16] bus/fslmc: verify strdup return value [04/16] bus/vdev: verify strdup return value [05/16] dma/idxd: verify strdup return value [06/16] event/cnxk: verify strdup return value [07/16] net/failsafe: fix memory leak when parse args [08/16] net/nfp: verify strdup return value [09/16] app/dumpcap: verify strdup return value [10/16] app/pdump: verify strdup return value [11/16] app/test: verify strdup return value [12/16] app/test-crypto-perf: verify strdup return value [13/16] app/test-dma-perf: verify strdup return value [14/16] app/testpmd: verify strdup return value [15/16] examples/qos_sched: fix memory leak when parse args [16/16] examples/vhost: verify strdup return value

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

fengchengwen Nov. 10, 2023, 6:36 a.m. UTC

  Add verify strdup return value logic.

Fixes: 64051bb1f144 ("devargs: unify scratch buffer storage")
Cc: stable@dpdk.org

Signed-off-by: Chengwen Feng <fengchengwen@huawei.com>
---
 drivers/bus/vdev/vdev.c | 4 ++++
 1 file changed, 4 insertions(+)

diff mbox series

Patch

diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c
index 7974b27295..05582f1727 100644
--- a/drivers/bus/vdev/vdev.c
+++ b/drivers/bus/vdev/vdev.c
@@ -247,6 +247,10 @@  alloc_devargs(const char *name, const char *args)
 		devargs->data = strdup(args);
 	else
 		devargs->data = strdup("");
+	if (devargs->data == NULL) {
+		free(devargs);
+		return NULL;
+	}
 	devargs->args = devargs->data;
 
 	ret = strlcpy(devargs->name, name, sizeof(devargs->name));