diff options
author | Sorin Vinturis <svinturis@cloudbasesolutions.com> | 2015-07-01 20:03:01 +0000 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2015-07-01 16:24:04 -0700 |
commit | d30fa82cd4211d66bab248370d84da803f181e44 (patch) | |
tree | 51996a46638c504e99afafaa700e05ea3968dfde /datapath-windows/ovsext | |
parent | 35303d715b1f0db46e6a27146815061a60385dc6 (diff) | |
download | openvswitch-d30fa82cd4211d66bab248370d84da803f181e44.tar.gz |
datapath-windows: Solved BSOD when handling flows
OvsPrepareFlow() returns an error only when the new flow allocation
fails. In this case HandleFlowPut() should return error without trying
to free the flow, thus avoiding the BSOD.
Signed-off-by: Sorin Vinturis <svinturis@cloudbasesolutions.com>
Reported-by: Sorin Vinturis <svinturis@cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/91
Acked-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'datapath-windows/ovsext')
-rw-r--r-- | datapath-windows/ovsext/Flow.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/datapath-windows/ovsext/Flow.c b/datapath-windows/ovsext/Flow.c index 6fa10a33f..b93f475df 100644 --- a/datapath-windows/ovsext/Flow.c +++ b/datapath-windows/ovsext/Flow.c @@ -2163,7 +2163,6 @@ HandleFlowPut(OvsFlowPut *put, status = OvsPrepareFlow(&KernelFlow, put, hash); if (status != STATUS_SUCCESS) { - FreeFlow(KernelFlow); return STATUS_UNSUCCESSFUL; } |