aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPravin B Shelar <pshelar@nicira.com>2014-11-11 16:40:49 -0500
committerPravin B Shelar <pshelar@nicira.com>2014-11-14 18:13:26 -0500
commitab64f16ff2e83371927c57a0380fd3c0fee5c1c1 (patch)
tree0fc4278aa8fe57325a1e6ccda16efa76182f3fcd
parentb23dc5a7cc6ebc9a0d57351da7a0e8454c9ffea3 (diff)
openvswitch: Fix memory leak.
Need to free memory in case of sample action error. Introduced by commit 651887b0c22cffcfce7eb9c ("openvswitch: Sample action without side effects"). Signed-off-by: Pravin B Shelar <pshelar@nicira.com>
-rw-r--r--net/openvswitch/actions.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/net/openvswitch/actions.c b/net/openvswitch/actions.c
index 006886dbee36..00e447a17f64 100644
--- a/net/openvswitch/actions.c
+++ b/net/openvswitch/actions.c
@@ -722,8 +722,6 @@ static int do_execute_actions(struct datapath *dp, struct sk_buff *skb,
722 722
723 case OVS_ACTION_ATTR_SAMPLE: 723 case OVS_ACTION_ATTR_SAMPLE:
724 err = sample(dp, skb, key, a); 724 err = sample(dp, skb, key, a);
725 if (unlikely(err)) /* skb already freed. */
726 return err;
727 break; 725 break;
728 } 726 }
729 727