Hi Monjalon,
This fix only moves the memcpy after to null check.
so the misplaced of memcpy and null check is reordered.
I thought maybe it's too trival a fix to add more explanation.
Sorry if it makes any confusion.
(last mail is garbled in format)
-Weiguo
> -----Original Message-----
> From: "Thomas Monjalon" <tho...@monjalon.net>;
> Date: Wed, Feb 2, 2022 07:50 PM
> To: "John Daley (johndale)"<johnd...@cisco.com>;"Weiguo
> Li"<liw...@foxmail.com>;
> Cc: "dev"<dev@dpdk.org>;"Ferruh Yigit"<ferruh.yi...@intel.com>;
> Subject: Re: [PATCH] net/enic: adjust memory check and use in proper order
>
> > > Subject: [PATCH] net/enic: adjust memory check and use in proper order
> > >
> > > Fixes: bb66d562aefc ("net/enic: share flow actions with same signature")
> > >
> > > Signed-off-by: Weiguo Li <liw...@foxmail.com>
> > >
> > > Reviewed-by: John Daley <johnd...@cisco.com>
> > >
> > > Applied to dpdk-next-net/main, thanks.
> Please could we have a little of explanation in the commit log?
> The title says "adjust [...] and use in proper order"
> but I see only a move of the check.
