diff options
| author | Alin Serdean <aserdean@cloudbasesolutions.com> | 2017-07-14 04:40:56 +0000 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2017-08-02 11:30:50 -0700 |
| commit | 0d4374aa82dea2dc316c432c0cfee371456fcf9d (patch) | |
| tree | d5a47992fb44e5d14f12859d498388d8ad0ca0dd /datapath-windows/ovsext/BufferMgmt.c | |
| parent | b27b4e5f05a6f2e27bd5ef9b0078a5ccc6678b7c (diff) | |
| download | openvswitch-0d4374aa82dea2dc316c432c0cfee371456fcf9d.tar.gz | |
datapath-windows: Add assert in OvsPartialCopyNBL
`srcNb` should never be NULL since it was copied over from another nbl.
Add an assertion just in case and to keep static analysis happy.
Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Shashank Ram <rams@vmware.com>
Diffstat (limited to 'datapath-windows/ovsext/BufferMgmt.c')
| -rw-r--r-- | datapath-windows/ovsext/BufferMgmt.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/datapath-windows/ovsext/BufferMgmt.c b/datapath-windows/ovsext/BufferMgmt.c index 6e4cfc535..13eaf9529 100644 --- a/datapath-windows/ovsext/BufferMgmt.c +++ b/datapath-windows/ovsext/BufferMgmt.c @@ -802,6 +802,7 @@ OvsPartialCopyNBL(PVOID ovsContext, OVS_BUFFER_PRIVATE_FORWARD_CONTEXT; srcNb = NET_BUFFER_LIST_FIRST_NB(nbl); + ASSERT(srcNb); OvsInitNBLContext(dstCtx, flags, NET_BUFFER_DATA_LENGTH(srcNb) - copySize, OVS_DPPORT_NUMBER_INVALID); |