Acked-by: Sairam Venugopal <vsai...@vmware.com>
On 10/27/16, 1:45 PM, "Sairam Venugopal" <vsai...@vmware.com> wrote: >Acked-by: Sairam Venugopal <vsai...@vmware.com> > > >On 10/27/16, 11:57 AM, "Alin Serdean" <aserd...@cloudbasesolutions.com> >wrote: > >>'status' is of type 'NTSTATUS' and NlFillOvsMsgForNfGenMsg is of type >>bool. >> >>Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com> >>--- >> datapath-windows/ovsext/Conntrack.c | 14 ++++++++------ >> 1 file changed, 8 insertions(+), 6 deletions(-) >> >>diff --git a/datapath-windows/ovsext/Conntrack.c >>b/datapath-windows/ovsext/Conntrack.c >>index 74fb38c..d81ef13 100644 >>--- a/datapath-windows/ovsext/Conntrack.c >>+++ b/datapath-windows/ovsext/Conntrack.c >>@@ -744,12 +744,14 @@ OvsCtDeleteCmdHandler(POVS_USER_PARAMS_CONTEXT >>usrParamsCtx, >> NlBufInit(&nlBuf, >> usrParamsCtx->outputBuffer, >> usrParamsCtx->outputLength); >>- status = NlFillOvsMsgForNfGenMsg(&nlBuf, nlmsgType, >>NLM_F_CREATE, >>- msgIn->nlMsg.nlmsgSeq, >>- msgIn->nlMsg.nlmsgPid, >>- AF_UNSPEC, >>- msgIn->nfGenMsg.version, >>- 0); >>+ if (!NlFillOvsMsgForNfGenMsg(&nlBuf, nlmsgType, NLM_F_CREATE, >>+ msgIn->nlMsg.nlmsgSeq, >>+ msgIn->nlMsg.nlmsgPid, >>+ AF_UNSPEC, >>+ msgIn->nfGenMsg.version, >>+ 0)) { >>+ status = STATUS_INVALID_PARAMETER; >>+ } >> nlMsg = (PNL_MSG_HDR)NlBufAt(&nlBuf, 0, 0); >> nlMsg->nlmsgLen = NlBufSize(&nlBuf); >> *replyLen = msgOut->nlMsg.nlmsgLen; >>-- >>2.9.2.windows.1 >>_______________________________________________ >>dev mailing list >>d...@openvswitch.org >>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailm >>a >>n_listinfo_dev&d=CwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=D >>c >>ruz40PROJ40ROzSpxyQSLw6fcrOWpJgEcEmNR3JEQ&m=1Y2zDajaljmsOhZZfX-BX6qooMBLZ >>s >>9FryoNHjBA2gU&s=H8Etjk7iamWO8ZH7cbxYoSJVlPp_XmIIqSCMz1_yTnQ&e= > >_______________________________________________ >dev mailing list >d...@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=NArguAsOJYEA5zKykHa8AIsQN2OLim >nNolWVlt4YBlM&s=Xr_n64qspeO9Dg1NQtN0xmDo-CYfh4fR9Os9vz14eXY&e= _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev