mirror of
https://github.com/qemu/qemu.git
synced 2025-08-02 12:45:00 +00:00
![]() Coverity points out that in ctucan_send_ready_buffers() we set buff_st_mask = 0xf << (i * 4) inside the loop, but then we never use it before overwriting it later. The only thing we use the mask for is as part of the code that is inserting the new buff_st field into tx_status. That is more comprehensibly written using deposit32(), so do that and drop the mask variable entirely. We also update the buff_st local variable at multiple points during this function, but nothing can ever see these intermediate values, so just drop those, write the final TXT_TOK as a fixed constant value, and collapse the only remaining set/use of buff_st down into an extract32(). Fixes: Coverity CID 1432869 Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Acked-by: Pavel Pisa <pisa@cmp.felk.cvut.cz> Tested-by: Pavel Pisa <pisa@cmp.felk.cvut.cz> Signed-off-by: Jason Wang <jasowang@redhat.com> |
||
---|---|---|
.. | ||
can_kvaser_pci.c | ||
can_mioe3680_pci.c | ||
can_pcm3680_pci.c | ||
can_sja1000.c | ||
can_sja1000.h | ||
ctu_can_fd_frame.h | ||
ctu_can_fd_regs.h | ||
ctucan_core.c | ||
ctucan_core.h | ||
ctucan_pci.c | ||
meson.build |