Acked-by: Sairam Venugopal <[email protected]>
On 10/27/16, 11:57 AM, "Alin Serdean" <[email protected]> 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 <[email protected]> >--- > 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 >[email protected] >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 [email protected] http://openvswitch.org/mailman/listinfo/dev
