RE: [PATCH v2 03/16] bus/fslmc: verify strdup return value

[Sachin Saxena]

Acked-by: Sachin Saxena <sachin.sax...@nxp.com>

> -----Original Message-----
> From: Chengwen Feng <fengcheng...@huawei.com>
> Sent: 10 November 2023 03:31 PM
> To: tho...@monjalon.net; ferruh.yi...@amd.com; Hemant Agrawal
> <hemant.agra...@nxp.com>; Sachin Saxena <sachin.sax...@nxp.com>;
> Shreyansh Jain <shreyansh.j...@nxp.com>
> Cc: dev@dpdk.org
> Subject: [PATCH v2 03/16] bus/fslmc: verify strdup return value
> 
> Add verify strdup return value logic.
> 
> Fixes: e67a61614d0b ("bus/fslmc: support device iteration")
> Cc: sta...@dpdk.org
> 
> Signed-off-by: Chengwen Feng <fengcheng...@huawei.com>
> ---
>  drivers/bus/fslmc/fslmc_bus.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/bus/fslmc/fslmc_bus.c b/drivers/bus/fslmc/fslmc_bus.c
> index 57bfb5111a..89f0f329c0 100644
> --- a/drivers/bus/fslmc/fslmc_bus.c
> +++ b/drivers/bus/fslmc/fslmc_bus.c
> @@ -634,6 +634,10 @@ fslmc_bus_dev_iterate(const void *start, const char
> *str,
> 
>       /* Now that name=device_name format is available, split */
>       dup = strdup(str);
> +     if (dup == NULL) {
> +             DPAA2_BUS_DEBUG("Dup string (%s) failed!\n", str);
> +             return NULL;
> +     }
>       dev_name = dup + strlen("name=");
> 
>       if (start != NULL) {
> --
> 2.17.1