NlBufAt should be called with valid boundary limits (within head and tail).
Incorrect argument to NlBufAt was leading to assert hit, fixed the same.
Signed-off-by: Ankur Sharma <ankursha...@vmware.com>
Acked-by: Nithin Raju <nit...@vmware.com>
Tested-by: Nithin Raju <nit...@vmware.com>
---
datapath-windows/ovsext/Netlink/Netlink.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/datapath-windows/ovsext/Netlink/Netlink.c
b/datapath-windows/ovsext/Netlink/Netlink.c
index 7ec651f..ae10a87 100644
--- a/datapath-windows/ovsext/Netlink/Netlink.c
+++ b/datapath-windows/ovsext/Netlink/Netlink.c
@@ -51,7 +51,7 @@ NlFillOvsMsg(PNL_BUFFER nlBuf, UINT16 nlmsgType,
/* To keep compiler happy for release build. */
UNREFERENCED_PARAMETER(offset);
- ASSERT(NlBufAt(nlBuf, offset, sizeof(struct _OVS_MESSAGE)) != 0);
+ ASSERT(NlBufAt(nlBuf, offset, 0) != 0);
msgOut.nlMsg.nlmsgType = nlmsgType;
msgOut.nlMsg.nlmsgFlags = nlmsgFlags;
@@ -88,7 +88,7 @@ NlFillNlHdr(PNL_BUFFER nlBuf, UINT16 nlmsgType,
/* To keep compiler happy for release build. */
UNREFERENCED_PARAMETER(offset);
- ASSERT(NlBufAt(nlBuf, offset, sizeof(struct _NL_MSG_HDR)) != 0);
+ ASSERT(NlBufAt(nlBuf, offset, 0) != 0);
msgOut.nlmsgType = nlmsgType;
msgOut.nlmsgFlags = nlmsgFlags;
--
1.9.1
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
