Re: [dpdk-dev] [PATCH] hash: fix return value of null not checked

[wangyunjian]

> -----Original Message-----
> From: Wang, Yipeng1 [mailto:yipeng1.w...@intel.com]
> Sent: Wednesday, July 22, 2020 4:24 AM
> To: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>; wangyunjian
> <wangyunj...@huawei.com>; dev@dpdk.org
> Cc: Gobriel, Sameh <sameh.gobr...@intel.com>; Richardson, Bruce
> <bruce.richard...@intel.com>; Lilijun (Jerry) <jerry.lili...@huawei.com>;
> xudingke <xudin...@huawei.com>; sta...@dpdk.org; nd <n...@arm.com>
> Subject: RE: [dpdk-dev] [PATCH] hash: fix return value of null not checked
> 
> > -----Original Message-----
> > From: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
> > Sent: Tuesday, July 21, 2020 10:27 AM
> > To: wangyunjian <wangyunj...@huawei.com>; dev@dpdk.org
> > Cc: Wang, Yipeng1 <yipeng1.w...@intel.com>; Gobriel, Sameh
> > <sameh.gobr...@intel.com>; Richardson, Bruce
> > <bruce.richard...@intel.com>; jerry.lili...@huawei.com;
> > xudin...@huawei.com; sta...@dpdk.org; nd <n...@arm.com>; Honnappa
> > Nagarahalli <honnappa.nagaraha...@arm.com>; nd <n...@arm.com>
> > Subject: RE: [dpdk-dev] [PATCH] hash: fix return value of null not
> > checked
> >
> >
> >
> > > -----Original Message-----
> > > From: dev <dev-boun...@dpdk.org> On Behalf Of wangyunjian
> > > Sent: Tuesday, July 21, 2020 8:32 AM
> > > To: dev@dpdk.org
> > > Cc: yipeng1.w...@intel.com; sameh.gobr...@intel.com;
> > > bruce.richard...@intel.com; jerry.lili...@huawei.com;
> > > xudin...@huawei.com; Yunjian Wang <wangyunj...@huawei.com>;
> > > sta...@dpdk.org
> > > Subject: [dpdk-dev] [PATCH] hash: fix return value of null not
> > > checked
> > >
> > > From: Yunjian Wang <wangyunj...@huawei.com>
> > >
> > > The function rte_zmalloc_socket() could return NULL, the return
> > > value need to be checked.
> > >
> > > Fixes: 5915699153d7 ("hash: fix scaling by reducing contention")
> > > Cc: sta...@dpdk.org
> > >
> > > Reported-by: HuangBin <brian.huang...@huawei.com>
> [Wang, Yipeng] missed a space in name?

Thanks, I will send v2 with correction.

Yunjian

> 
> > > Signed-off-by: Yunjian Wang <wangyunj...@huawei.com>
> > Good catch
> > Reviewed-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
> [Wang, Yipeng]
> Thanks for the fix!
> 
> Acked-by: Yipeng Wang <yipeng1.w...@intel.com>