diff options
| author | Ankur Sharma <ankursharma@vmware.com> | 2014-10-11 15:07:41 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2014-10-13 14:23:08 -0700 |
| commit | 2aca2188999d8986b664d4813b88c5c5823dc977 (patch) | |
| tree | 85ce837047cf40ac43e350bd2c4d2ecc9a62ae70 /datapath-windows/ovsext/Netlink | |
| parent | c881751a15b6b8f17e1febd8cc86536948bc38a8 (diff) | |
| download | openvswitch-2aca2188999d8986b664d4813b88c5c5823dc977.tar.gz | |
datapath-windows: Fix assertion in Netlink.c
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 <ankursharma@vmware.com>
Acked-by: Nithin Raju <nithin@vmware.com>
Tested-by: Nithin Raju <nithin@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'datapath-windows/ovsext/Netlink')
| -rw-r--r-- | datapath-windows/ovsext/Netlink/Netlink.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/datapath-windows/ovsext/Netlink/Netlink.c b/datapath-windows/ovsext/Netlink/Netlink.c index 7ec651f08..ae10a870d 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; |