[master] 57cd74cf5 Make FlexeLint a little bit happier

Nils Goroll nils.goroll at uplex.de
Wed Nov 14 10:35:48 UTC 2018


On 14/11/2018 09:36, Poul-Henning Kamp wrote:
> +	if (args->valid_b)
> +		AN(args->b);

this is wrong semantically: If we have a boolean argument, that does not imply
that the boolean is true.

In contrast, if we have a string argument, it needs to be a non-NULL pointer

-- 

** * * UPLEX - Nils Goroll Systemoptimierung

Scheffelstraße 32
22301 Hamburg

tel +49 40 28805731
mob +49 170 2723133
fax +49 40 42949753

xmpp://slink@jabber.int.uplex.de/

http://uplex.de/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-commit/attachments/20181114/3524dd7d/attachment-0001.bin>


More information about the varnish-commit mailing list