Re: privoxy-3.0.31-stable/jcc.c:3944:25: style: Suspicious condition

Fabian Keil <[email protected]> Wed, 3 Feb 2021 12:04:25 +0100
Newsgroups gmane.comp.web.privoxy.user
Message-ID <[email protected]>
David Binderman <[email protected]> wrote on 2021-02-02:

> Some static analysis has produced this message:
> 
> privoxy-3.0.31-stable/jcc.c:3944:25: style: Suspicious condition
> (assignment + comparison); Clarify expression with parentheses.
> [clarifyCondition]
[...]
> Maybe better code
> 
>                      || ((len = ssl_flush_socket(&(csp->ssl_client_attr),
>                             csp->iob)) < 0))

Thanks for the report. I added the parentheses:
https://www.privoxy.org/gitweb/?p=privoxy.git;a=commitdiff;h=1735db35cf6e78e892535418accc59f23c335881

Fabian

_______________________________________________
Privoxy-users mailing list
[email protected]
https://lists.privoxy.org/mailman/listinfo/privoxy-users
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEETzbBfzgWkTZUoehQaRgikYuiNxwFAmAagzoACgkQaRgikYui
NxxxdQ/8DlLcDKhPD9y4cQjEpglZxWMR6kSIBfKTMldXZdyX5zAcBo50qf2AXZHG
y2ukhe8uKDmkwhqjt57N9LYROvG0tevKQcg42KRNpybv8L1V4g5kgKOvv5u8LRSa
uAOfPJ1IxzW6CXe3XAA/o6Z6+6s+/U8ItsDKKLcKGcUbdivxpin51NyDPFz7Yr8G
X5/IBexgv3cIdcp/fLlMo77HO1vWaLZvEB/gXq2NzeIwLXw+zYtY2ESI9Ab6aAhD
a5uM+CFvtYR/tQncxv1p+mwtN0Y1aKkgQz77pRQixGMp3DX6cK1f3ykUCX6FOCww
xK/KgZAh8tB2XpOuyxYcY+FsRI+wg7E9cJ3FSFzMgpqcBL1NStOmn+7ld36Rtz82
wWhW35pfEoRz2Yt1QsPYMgadAENkTNfyAucf8ezZRHyahg3dO45IAqNGCG+0N8FC
YvSLJDcG/04EHNqk71wovGhDthpDZhAF42pFwaP5DMyRSc2HenBnlx67dLkN+KtX
TE53xIMbc3mlAnDxUH81qITuATUY5/p2sG/GQaeemFSsmKhGydKSKS1b0uuPlWc1
hJ7IPgUTsrTNLLyi3XWHMtNBIFkqCenn6xlYoJCtjNBlP2TB69Qbjq/njyMyM8p7
lgxsyfdLvANyudG0wIN+Qj8XQdNAstzB0kaMbAvF/z2cuO2v7sA=
=VhhJ
-----END PGP SIGNATURE-----