diff options
author | Stefano Brivio <sbrivio@redhat.com> | 2023-02-27 02:45:42 +0100 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2023-02-27 18:54:38 +0100 |
commit | 5474bc5485d814acae19961f9a9cd4b541722a5e (patch) | |
tree | bbc27104cbf4a7ed970184a10b4178f7c59f2b4f /icmp.c | |
parent | a48c5c2abf8aba39c32cf6845d51b5ca05d33361 (diff) | |
download | passt-5474bc5485d814acae19961f9a9cd4b541722a5e.tar passt-5474bc5485d814acae19961f9a9cd4b541722a5e.tar.gz passt-5474bc5485d814acae19961f9a9cd4b541722a5e.tar.bz2 passt-5474bc5485d814acae19961f9a9cd4b541722a5e.tar.lz passt-5474bc5485d814acae19961f9a9cd4b541722a5e.tar.xz passt-5474bc5485d814acae19961f9a9cd4b541722a5e.tar.zst passt-5474bc5485d814acae19961f9a9cd4b541722a5e.zip |
tcp, tcp_splice: Get rid of false positive CWE-394 Coverity warning from fls()
We use the return value of fls() as array index for debug strings.
While fls() can return -1 (if no bit is set), Coverity Scan doesn't
see that we're first checking the return value of another fls() call
with the same bitmask, before using it.
Call fls() once, store its return value, check it, and use the stored
value as array index.
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'icmp.c')
0 files changed, 0 insertions, 0 deletions