Messages in this thread | | | Date | Thu, 24 Dec 2020 18:16:22 +0800 (GMT+08:00) | From | dinghao.liu@zju ... | Subject | Re: Re: [PATCH] enic: Remove redundant free in enic_set_ringparam |
| |
> On Wed, 23 Dec 2020 20:38:33 +0800 Dinghao Liu wrote: > > The error handling paths in enic_alloc_vnic_resources() > > have called enic_free_vnic_resources() before returning. > > So we may not need to call it again on failure at caller > > side. > > > > Signed-off-by: Dinghao Liu <dinghao.liu@zju.edu.cn> > > But it's harmless, right? So the patch is just a cleanup not a fix? >
I think it's harmless. Since there is a check every time before freeing, calling enic_free_vnic_resources() twice has no security issue.
> In that case, could you please repost in two weeks? We're currently > in the merge window period, we're only accepting fixes to the > networking trees. > > Thanks!
| |