diff options
| author |   Lukas Bulwahn <lukas.bulwahn@gmail.com> | 2020-10-28 12:35:33 +0100 |
|---|---|---|
| committer |   Jakub Kicinski <kuba@kernel.org> | 2020-10-30 14:05:55 -0700 |
| commit | 0ad41b244ca0ca07af2a652014bc929cbdf91e87 (patch) | |
| tree | 3b73d9f7a6cc649812dd0b48925773ac6c6ca063 /drivers/net/phy/marvell.c | |
| parent | ipv6: mcast: make annotations for ip6_mc_msfget() consistent (diff) | |
| download | wireguard-linux-0ad41b244ca0ca07af2a652014bc929cbdf91e87.tar.xz wireguard-linux-0ad41b244ca0ca07af2a652014bc929cbdf91e87.zip | |
net: cls_api: remove unneeded local variable in tc_dump_chain()
make clang-analyzer on x86_64 defconfig caught my attention with:
net/sched/cls_api.c:2964:3: warning: Value stored to 'parent' is never read
[clang-analyzer-deadcode.DeadStores]
parent = 0;
^
net/sched/cls_api.c:2977:4: warning: Value stored to 'parent' is never read
[clang-analyzer-deadcode.DeadStores]
parent = q->handle;
^
Commit 32a4f5ecd738 ("net: sched: introduce chain object to uapi")
introduced tc_dump_chain() and this initial implementation already
contained these unneeded dead stores.
Simplify the code to make clang-analyzer happy.
As compilers will detect these unneeded assignments and optimize this
anyway, the resulting binary is identical before and after this change.
No functional change. No change in object code.
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Link: https://lore.kernel.org/r/20201028113533.26160-1-lukas.bulwahn@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net/phy/marvell.c')
0 files changed, 0 insertions, 0 deletions