]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Make Coverity more happy with r334545
authorvangyzen <vangyzen@FreeBSD.org>
Tue, 5 Jun 2018 20:34:11 +0000 (20:34 +0000)
committervangyzen <vangyzen@FreeBSD.org>
Tue, 5 Jun 2018 20:34:11 +0000 (20:34 +0000)
commit515692db3e212a3c4dffb544fbdaa1ae7528c88e
tree5fb205e2f44d450040754a220c5319cdd9e54212
parent8e40219f08f268b9058b473a77b266b1b5a79976
Make Coverity more happy with r334545

Coverity complains about:

if (((flags) & M_WAITOK) || _malloc_item != NULL)

saying:

The expression
1 /* (2 | 0x100) & 2 */ || _malloc_item != NULL
is suspicious because it performs a Boolean operation
on a constant other than 0 or 1.

Although the code is correct, add "!= 0" to make it slightly
more legible and to silence hundreds(?) of Coverity warnings.

Reported by: Coverity
Discussed with: mjg
Sponsored by: Dell EMC
sys/sys/malloc.h