This fix only moves the memcpy after null check. 
so the misplaced of memcpy and null check is reordered.
I thought it's too trival a fix to add more explanation.


Sorry if makes any confusion.


-Weiguo


> -----Original Message-----
&gt; From:&nbsp;"Thomas Monjalon" <thomas@monjalon.net&gt;;
&gt; Date: Wed, Feb 2, 2022 07:50 PM
&gt; To: "John Daley (johndale)"<johndale@cisco.com&gt;;"Weiguo Li"<liwg06@foxmail.com&gt;;
&gt; Cc: "dev"<dev@dpdk.org&gt;;"Ferruh Yigit"<ferruh.yigit@intel.com&gt;;
&gt; Subject: Re: [PATCH] net/enic: adjust memory check and use in proper order
&gt;

&gt; &gt; &gt; Subject: [PATCH] net/enic: adjust memory check and use in proper order
&gt; &gt; &gt;
&gt; &gt;&nbsp;&gt;&nbsp;Fixes: bb66d562aefc ("net/enic: share flow actions with same signature")
&gt; &gt;&nbsp;&gt;
&gt; &gt;&nbsp;&gt;&nbsp;Signed-off-by: Weiguo Li <liwg06@foxmail.com&gt;
&gt; &gt;&nbsp;&gt;
&gt; &gt;&nbsp;&gt;&nbsp;Reviewed-by: John Daley <johndale@cisco.com&gt;
&gt; &gt;&nbsp;&gt;
&gt; &gt;&nbsp;&gt;&nbsp;Applied to dpdk-next-net/main, thanks.

&nbsp;&gt; Please could we have a little of explanation in the commit log?
&nbsp;&gt; The title says "adjust [...] and use in proper order"
&nbsp;&gt; but I see only a move of the check.