Acked-by: Sairam Venugopal <vsai...@vmware.com>
On 10/27/16, 11:55 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/Stt.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/datapath-windows/ovsext/Stt.c b/datapath-windows/ovsext/Stt.c >index ad322d2..b04a77f 100644 >--- a/datapath-windows/ovsext/Stt.c >+++ b/datapath-windows/ovsext/Stt.c >@@ -278,10 +278,10 @@ OvsDoEncapStt(POVS_VPORT_ENTRY vport, > sttHdr = (SttHdr *) (outerTcpHdr + 1); > > /* L2 header */ >- ASSERT(((PCHAR)&fwdInfo->dstMacAddr + sizeof >fwdInfo->dstMacAddr) == >- (PCHAR)&fwdInfo->srcMacAddr); > NdisMoveMemory(outerEthHdr->Destination, fwdInfo->dstMacAddr, >- sizeof outerEthHdr->Destination + sizeof >outerEthHdr->Source); >+ sizeof outerEthHdr->Destination); >+ NdisMoveMemory(outerEthHdr->Source, fwdInfo->srcMacAddr, >+ sizeof outerEthHdr->Source); > outerEthHdr->Type = htons(ETH_TYPE_IPV4); > > /* L3 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=lxpLbxUQzMheqMsqQHwHVlpVLpdmuX >iWgI-ntenuWM8&s=KazJ2v6b52_iR4Mzn-WV0PFVi_axdjOEQs82ekLxix8&e= _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev