diff options
| author | Alin Serdean <aserdean@cloudbasesolutions.com> | 2017-11-06 17:33:39 +0200 |
|---|---|---|
| committer | Alin Gabriel Serdean <aserdean@ovn.org> | 2017-11-29 16:51:19 +0200 |
| commit | bf7e459bf4d216678d2e68a607141eb62f5649af (patch) | |
| tree | f99c0b8fbbd83e91f0df533578187aeb1f502f3c /datapath-windows | |
| parent | 7d20979a88b37c580346d278663cde0d4c78dae9 (diff) | |
| download | openvswitch-bf7e459bf4d216678d2e68a607141eb62f5649af.tar.gz | |
datapath-windows: Fix possible NULL dereference in IpFragment
If we can't allocate the NBL just go to the cleanup sequence.
Found using WDK 10 static code analysis.
Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Acked-by: Shashank Ram <shashank08@gmail.com>
Diffstat (limited to 'datapath-windows')
| -rw-r--r-- | datapath-windows/ovsext/IpFragment.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/datapath-windows/ovsext/IpFragment.c b/datapath-windows/ovsext/IpFragment.c index ad48834a6..3d5277ac4 100644 --- a/datapath-windows/ovsext/IpFragment.c +++ b/datapath-windows/ovsext/IpFragment.c @@ -206,6 +206,7 @@ OvsIpv4Reassemble(POVS_SWITCH_CONTEXT switchContext, if (*newNbl == NULL) { OVS_LOG_ERROR("Insufficient resources, failed to allocate newNbl"); status = NDIS_STATUS_RESOURCES; + goto cleanup; } /* Complete the fragment NBL */ |