<meta http-equiv="Content-Type" content="text/html; charset=GB18030"><div>This fix only moves the memcpy after null check. </div><div><div>so the misplaced of memcpy and null check is reordered.</div><div>I thought it's too trival a fix to add more explanation.</div><div><br></div><div>Sorry if makes any confusion.</div><div><br></div><div>-Weiguo</div><div><br></div><font editorwarp__="1" style="display: inline; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;"><div style="font-size: 14px; font-family: Verdana; padding: 2px 0px; display: block; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;">> -----Original Message-----</div><div style="font-size: 14px; font-family: Verdana; padding: 2px 0px; display: block; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;"><span style="background-color: rgba(0, 0, 0, 0);">> </span><span style="background-color: rgba(0, 0, 0, 0);">From:</span><span style="background-color: rgba(0, 0, 0, 0);"> </span><span style="background-color: rgba(0, 0, 0, 0);">"Thomas Monjalon" <thomas@monjalon.net>;</span></div><div style="display: block; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;"><div style="background-color: rgba(0, 0, 0, 0);">> Date: Wed, Feb 2, 2022 07:50 PM</div><div style="background-color: rgba(0, 0, 0, 0);">> To: "John Daley (johndale)"<johndale@cisco.com>;"Weiguo Li"<liwg06@foxmail.com>;</div><div style="background-color: rgba(0, 0, 0, 0);">> Cc: "dev"<dev@dpdk.org>;"Ferruh Yigit"<ferruh.yigit@intel.com>;</div><div style="background-color: rgba(0, 0, 0, 0);">> Subject: Re: [PATCH] net/enic: adjust memory check and use in proper order</div><div style="background-color: rgba(0, 0, 0, 0);">></div></div>> > </font><font editorwarp__="1" style="display: inline; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;">> Subject: [PATCH] net/enic: adjust memory check and use in proper order</font></div><font editorwarp__="1" style="display: inline; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;"><div style="display: block; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;">> > ><br>> ><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;"> </font><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;">></font><span style="background-color: rgba(0, 0, 0, 0);"> Fixes: bb66d562aefc ("net/enic: share flow actions with same signature")</span></div><div style="display: block; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;">> ><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;"> </font><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;">></font><br>> ><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;"> </font><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;">></font><span style="background-color: rgba(0, 0, 0, 0);"> Signed-off-by: Weiguo Li <liwg06@foxmail.com></span></div><div style="display: block; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;">> ><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;"> </font><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;">></font><br>> ><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;"> </font><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;">></font><span style="background-color: rgba(0, 0, 0, 0);"> Reviewed-by: John Daley <johndale@cisco.com></span></div><div style="display: block; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;">> ><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;"> </font><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;">></font><br>> ><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;"> </font><font editorwarp__="1" style="background-color: rgba(0, 0, 0, 0); display: inline;">></font><span style="background-color: rgba(0, 0, 0, 0);"> Applied to dpdk-next-net/main, thanks.</span></div><div style="display: block; font-size: 14px; font-family: Verdana; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0); font-weight: 400; font-style: normal;"><br> > Please could we have a little of explanation in the commit log?<br> > The title says "adjust [...] and use in proper order"<br> > but I see only a move of the check.</div></font><div><br><br></div>