[dpdk-dev] [PATCH] eal: Fix wrong error checking of rte_eal_parse_devargs_str()
On 2016/06/15 23:48, David Marchand wrote: > On Wed, Jun 15, 2016 at 5:26 AM, Tetsuya Mukawa wrote: >> Currently, a return value of strdup() isn't checked correctly. >> The patch fixes it. >> >> Signed-off-by: Tetsuya Mukawa > > Idem, the title does not need the function name to describe what is going > wrong. > > This might deserve a fixline, I traced this to 0fe11ec592b2 ("eal: add > vdev init and uninit"). Will add it. Thanks, Tetsuya
[dpdk-dev] [PATCH] eal: Fix wrong error checking of rte_eal_parse_devargs_str()
On Wed, Jun 15, 2016 at 5:26 AM, Tetsuya Mukawa wrote: > Currently, a return value of strdup() isn't checked correctly. > The patch fixes it. > > Signed-off-by: Tetsuya Mukawa Idem, the title does not need the function name to describe what is going wrong. This might deserve a fixline, I traced this to 0fe11ec592b2 ("eal: add vdev init and uninit"). > --- > lib/librte_eal/common/eal_common_devargs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/librte_eal/common/eal_common_devargs.c > b/lib/librte_eal/common/eal_common_devargs.c > index 2bfe54a..e403717 100644 > --- a/lib/librte_eal/common/eal_common_devargs.c > +++ b/lib/librte_eal/common/eal_common_devargs.c > @@ -58,7 +58,7 @@ rte_eal_parse_devargs_str(const char *devargs_str, > return -1; > > *drvname = strdup(devargs_str); > - if (drvname == NULL) > + if (*drvname == NULL) > return -1; > > /* set the first ',' to '\0' to split name and arguments */ > -- > 2.7.4 > -- David Marchand
[dpdk-dev] [PATCH] eal: Fix wrong error checking of rte_eal_parse_devargs_str()
Currently, a return value of strdup() isn't checked correctly. The patch fixes it. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_devargs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_eal/common/eal_common_devargs.c b/lib/librte_eal/common/eal_common_devargs.c index 2bfe54a..e403717 100644 --- a/lib/librte_eal/common/eal_common_devargs.c +++ b/lib/librte_eal/common/eal_common_devargs.c @@ -58,7 +58,7 @@ rte_eal_parse_devargs_str(const char *devargs_str, return -1; *drvname = strdup(devargs_str); - if (drvname == NULL) + if (*drvname == NULL) return -1; /* set the first ',' to '\0' to split name and arguments */ -- 2.7.4