On 3 November 2016 at 16:27, Sairam Venugopal <vsai...@vmware.com> wrote:
> Acked-by: Sairam Venugopal <vsai...@vmware.com> > Applied the first 4 patches of the series. Thanks! > > > 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 > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
