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 <svintu...@cloudbasesolutions.com> Reported-by: Sorin Vinturis <svintu...@cloudbasesolutions.com> Reported-at: https://github.com/openvswitch/ovs-issues/issues/91 Acked-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com> --- This patch should be applied both on master and branch 2.4. --- datapath-windows/ovsext/Flow.c | 1 - 1 file changed, 1 deletion(-) diff --git a/datapath-windows/ovsext/Flow.c b/datapath-windows/ovsext/Flow.c index 6fa10a3..b93f475 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; } -- 1.9.0.msysgit.0 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
