Acked-by: Sairam Venugopal <vsai...@vmware.com>
On 10/27/16, 11:57 AM, "Alin Serdean" <aserd...@cloudbasesolutions.com> wrote: >The purpose of this patch is to make the code more readable and >fix a static analyzer warning. > >Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com> >--- > datapath-windows/ovsext/Geneve.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/datapath-windows/ovsext/Geneve.c >b/datapath-windows/ovsext/Geneve.c >index 5712e4d..efdf9f7 100644 >--- a/datapath-windows/ovsext/Geneve.c >+++ b/datapath-windows/ovsext/Geneve.c >@@ -170,10 +170,10 @@ NDIS_STATUS OvsEncapGeneve(POVS_VPORT_ENTRY vport, > > /* L2 header */ > ethHdr = (EthHdr *)bufferStart; >- ASSERT(((PCHAR)&fwdInfo.dstMacAddr + sizeof fwdInfo.dstMacAddr) >== >- (PCHAR)&fwdInfo.srcMacAddr); > NdisMoveMemory(ethHdr->Destination, fwdInfo.dstMacAddr, >- sizeof ethHdr->Destination + sizeof >ethHdr->Source); >+ sizeof ethHdr->Destination); >+ NdisMoveMemory(ethHdr->Source, fwdInfo.srcMacAddr, >+ sizeof ethHdr->Source); > ethHdr->Type = htons(ETH_TYPE_IPV4); > > /* IP header */ >-- >2.9.2.windows.1 >_______________________________________________ >dev mailing list >dev@openvswitch.org >https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailma >n_listinfo_dev&d=CwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=Dc >ruz40PROJ40ROzSpxyQSLw6fcrOWpJgEcEmNR3JEQ&m=eLFTV4YDHXlRcAzrUIHZJVdeHQCkIq >Jpaqd-3rY_N40&s=rMgWehrHeV8ZOAxTok8cLkUXXwTWYom8ySBbCfBbymY&e= _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
