diff options
author | Alin Serdean <aserdean@cloudbasesolutions.com> | 2017-05-17 13:54:55 +0000 |
---|---|---|
committer | Gurucharan Shetty <guru@ovn.org> | 2017-05-23 15:29:57 -0700 |
commit | 42409aae15bab086462a5cb8dd22e4254ae59109 (patch) | |
tree | 98a64a873072ed94eda9106efbb8dd0f95f5e7f1 /datapath-windows/ovsext | |
parent | aa341b9f8667cc599d4513da023d01fab3f52f45 (diff) | |
download | openvswitch-42409aae15bab086462a5cb8dd22e4254ae59109.tar.gz |
datapath-windows: Fix possible null dereference in ipfragment
Found using static analysis tools.
Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Acked-by: Anand Kumar <kumaranand@vmware.com>
Signed-off-by: Gurucharan Shetty <guru@ovn.org>
Diffstat (limited to 'datapath-windows/ovsext')
-rw-r--r-- | datapath-windows/ovsext/IpFragment.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/datapath-windows/ovsext/IpFragment.c b/datapath-windows/ovsext/IpFragment.c index 675c32e5a..0874cb99d 100644 --- a/datapath-windows/ovsext/IpFragment.c +++ b/datapath-windows/ovsext/IpFragment.c @@ -343,7 +343,7 @@ OvsProcessIpv4Fragment(POVS_SWITCH_CONTEXT switchContext, } POVS_FRAGMENT_LIST next = entry->head; POVS_FRAGMENT_LIST prev = entry->tail; - if (prev != NULL || prev->offset < offset) { + if (prev != NULL && prev->offset < offset) { next = NULL; goto found; } |