diff options
author | Tom Rix | 2022-03-26 09:03:06 -0700 |
---|---|---|
committer | David S. Miller | 2022-03-26 14:14:04 -0700 |
commit | 33b5bc9e703383e396f275d51fc4bafa48dbae5a (patch) | |
tree | 5f3c46b51de4db96c10ee60c84508968f4d6f400 /tools/testing/cxl | |
parent | 0906f3a3df07835e37077d8971aac65347f2ed57 (diff) |
octeontx2-af: initialize action variable
Clang static analysis reports this representative issue
rvu_npc.c:898:15: warning: Assigned value is garbage
or undefined
req.match_id = action.match_id;
^ ~~~~~~~~~~~~~~~
The initial setting of action is conditional on
if (is_mcam_entry_enabled(...))
The later check of action.op will sometimes be garbage.
So initialize action.
Reduce setting of
*(u64 *)&action = 0x00;
to
*(u64 *)&action = 0;
Fixes: 967db3529eca ("octeontx2-af: add support for multicast/promisc packet replication feature")
Signed-off-by: Tom Rix <trix@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools/testing/cxl')
0 files changed, 0 insertions, 0 deletions