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" <[email protected]>;
> Date: Wed, Feb 2, 2022 07:50 PM
> To: "John Daley (johndale)"<[email protected]>;"Weiguo 
> Li"<[email protected]>;
> Cc: "dev"<[email protected]>;"Ferruh Yigit"<[email protected]>;
> 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 <[email protected]>
> > >
> > > Reviewed-by: John Daley <[email protected]>
> > >
> > > 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.

Reply via email to