diff options
author | Pravin B Shelar <pshelar@nicira.com> | 2014-11-11 16:40:49 -0500 |
---|---|---|
committer | Pravin B Shelar <pshelar@nicira.com> | 2014-11-14 18:13:26 -0500 |
commit | ab64f16ff2e83371927c57a0380fd3c0fee5c1c1 (patch) | |
tree | 0fc4278aa8fe57325a1e6ccda16efa76182f3fcd /net/openvswitch/actions.c | |
parent | b23dc5a7cc6ebc9a0d57351da7a0e8454c9ffea3 (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>
Diffstat (limited to 'net/openvswitch/actions.c')
-rw-r--r-- | net/openvswitch/actions.c | 2 |
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 | ||