diff options
author | Dan Carpenter | 2023-10-19 10:32:27 -0700 |
---|---|---|
committer | David S. Miller | 2023-10-20 12:53:06 +0100 |
commit | a41654c3ed1d7d6a2f7ae485f3e3c655b428f38f (patch) | |
tree | a787c5a354d729d3c81d1578c94cf855cc214dba /tools | |
parent | 4690aea589e76e887a40195b35b6d19a838058eb (diff) |
ixgbe: fix end of loop test in ixgbe_set_vf_macvlan()
The list iterator in a list_for_each_entry() loop can never be NULL.
If the loop exits without hitting a break then the iterator points
to an offset off the list head and dereferencing it is an out of
bounds access.
Before we transitioned to using list_for_each_entry() loops, then
it was possible for "entry" to be NULL and the comments mention
this. I have updated the comments to match the new code.
Fixes: c1fec890458a ("ethernet/intel: Use list_for_each_entry() helper")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Simon Horman <horms@kernel.org>
Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Tested-by: Rafal Romanowski <rafal.romanowski@intel.com>
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions