We should be returning value of tail before the increment and not after. Signed-off-by: Nithin Raju <nit...@vmware.com> --- datapath-windows/ovsext/Netlink/NetlinkBuf.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/datapath-windows/ovsext/Netlink/NetlinkBuf.c b/datapath-windows/ovsext/Netlink/NetlinkBuf.c index ba10d83..149bfb3 100644 --- a/datapath-windows/ovsext/Netlink/NetlinkBuf.c +++ b/datapath-windows/ovsext/Netlink/NetlinkBuf.c @@ -215,13 +215,11 @@ NlBufCopyAtTailUninit(PNL_BUFFER nlBuf, UINT32 len) { PCHAR ret = NULL; + ret = nlBuf->tail; if ((NlBufCopyAtTail(nlBuf, NULL, len)) == FALSE) { - goto done; + return NULL; } - ret = nlBuf->tail; - -done: return ret; } -- 1.7.4.1 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev