diff options
author | Jacob Keller <[email protected]> | 2017-02-06 14:38:41 -0800 |
---|---|---|
committer | Jeff Kirsher <[email protected]> | 2017-03-20 16:45:20 -0700 |
commit | e5187ee3ee9a95b1aa32a9a3daf4f07a9f89c20c (patch) | |
tree | 6607861cacb221079be19002d472235f2a0c43ef /tools/perf/util/c++/clang.cpp | |
parent | 01016da1e58136518252822738fe833c662df916 (diff) |
i40e: return immediately when failing to add fdir filter
Instead of setting err=true and checking this to determine when to free
the raw_packet near the end of the function, simply kfree and return
immediately. The resulting code is a bit cleaner and has one less
variable. This also resolves a subtle bug in the ipv4 case which could
fail to add the first filter and then never free the memory, resulting
in a small memory leak.
Change-ID: I7583aac033481dc794b4acaa14445059c8930ff1
Signed-off-by: Jacob Keller <[email protected]>
Reviewed-by: Avinash Dayanand <[email protected]>
Reviewed-by: Alan Brady <[email protected]>
Reviewed-by: Mitch Williams <[email protected]>
Tested-by: Andrew Bowers <[email protected]>
Signed-off-by: Jeff Kirsher <[email protected]>
Diffstat (limited to 'tools/perf/util/c++/clang.cpp')
0 files changed, 0 insertions, 0 deletions