diff mbox series

[3/9] compressdev: fix null name when parse input args

Message ID	20230302075012.32423-4-fengchengwen@huawei.com (mailing list archive)
State	Accepted, archived
Delegated to:	David Marchand
Headers	From: Chengwen Feng <fengchengwen@huawei.com> To: <thomas@monjalon.net>, <ferruh.yigit@amd.com>, Fan Zhang <fanzhang.oss@gmail.com>, Ashish Gupta <ashish.gupta@marvell.com>, Shally Verma <shallyv@marvell.com>, Fiona Trahe <fiona.trahe@intel.com>, Pablo de Lara <pablo.de.lara.guarch@intel.com> CC: <dev@dpdk.org> Subject: [PATCH 3/9] compressdev: fix null name when parse input args Date: Thu, 2 Mar 2023 07:50:06 +0000 Message-ID: <20230302075012.32423-4-fengchengwen@huawei.com> In-Reply-To: <20230302075012.32423-1-fengchengwen@huawei.com> References: <20230302075012.32423-1-fengchengwen@huawei.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: list Errors-To: dev-bounces@dpdk.org
Series	fix kvargs callback prototype not clearly defined \| [0/9] fix kvargs callback prototype not clearly defined [1/9] kvargs: detailed definition of callback prototype [2/9] compressdev: fix segment fault when parse input args [3/9] compressdev: fix null name when parse input args [4/9] cryptodev: fix segment fault when parse input args [5/9] cryptodev: fix null name when parse input args [6/9] net/hns3: fix segment fault when parse runtime config [7/9] net/virtio: fix segment fault when parse devargs [8/9] dma/skeleton: fix segment fault when parse devargs [9/9] raw/skeleton: fix segment fault when parse devargs

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

fengchengwen March 2, 2023, 7:50 a.m. UTC

  The rte_kvargs_process() was used to parse KV pairs, it also supports
to parse 'only keys' (e.g. socket_id) type. And the callback function
parameter 'value' is NULL when parsed 'only keys'.

If the input args is just "name" (which mean only has keys), then it
will get (null) name in rte_compressdev_pmd_parse_name_arg(). This
patch fixes it.

Fixes: ed7dd94f7f66 ("compressdev: add basic device management")
Cc: stable@dpdk.org

Signed-off-by: Chengwen Feng <fengchengwen@huawei.com>
---
 lib/compressdev/rte_compressdev_pmd.c | 3 +++
 1 file changed, 3 insertions(+)

diff mbox series

Patch

diff --git a/lib/compressdev/rte_compressdev_pmd.c b/lib/compressdev/rte_compressdev_pmd.c
index 93c31fbac8..156bccd972 100644
--- a/lib/compressdev/rte_compressdev_pmd.c
+++ b/lib/compressdev/rte_compressdev_pmd.c
@@ -23,6 +23,9 @@  rte_compressdev_pmd_parse_name_arg(const char *key __rte_unused,
 	struct rte_compressdev_pmd_init_params *params = extra_args;
 	int n;
 
+	if (value == NULL || extra_args == NULL)
+		return -EINVAL;
+
 	n = strlcpy(params->name, value, RTE_COMPRESSDEV_NAME_MAX_LEN);
 	if (n >= RTE_COMPRESSDEV_NAME_MAX_LEN)
 		return -EINVAL;