modify parentheses in conditionals to make it clearer what is being

assigned and what is being checked

ok djm dtucker

diff --git a/usr.bin/ssh/authfd.c b/usr.bin/ssh/authfd.c
index e365c90..4e67631 100644 (file)
--- a/usr.bin/ssh/authfd.c
+++ b/usr.bin/ssh/authfd.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: authfd.c,v 1.132 2023/03/09 06:58:26 djm Exp $ */
+/* $OpenBSD: authfd.c,v 1.133 2023/03/09 21:06:24 jcs Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -489,8 +489,8 @@ encode_dest_constraint(struct sshbuf *m, const struct dest_constraint *dc)
 
        if ((b = sshbuf_new()) == NULL)
                return SSH_ERR_ALLOC_FAIL;
-       if ((r = encode_dest_constraint_hop(b, &dc->from) != 0) ||
-           (r = encode_dest_constraint_hop(b, &dc->to) != 0) ||
+       if ((r = encode_dest_constraint_hop(b, &dc->from)) != 0 ||
+           (r = encode_dest_constraint_hop(b, &dc->to)) != 0 ||
            (r = sshbuf_put_string(b, NULL, 0)) != 0) /* reserved */
                goto out;
        if ((r = sshbuf_put_stringb(m, b)) != 0)

diff --git a/usr.bin/ssh/readconf.c b/usr.bin/ssh/readconf.c
index e9d3a75..6d29405 100644 (file)
--- a/usr.bin/ssh/readconf.c
+++ b/usr.bin/ssh/readconf.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: readconf.c,v 1.373 2023/03/05 05:34:09 dtucker Exp $ */
+/* $OpenBSD: readconf.c,v 1.374 2023/03/09 21:06:24 jcs Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -602,7 +602,7 @@ match_cfg_line(Options *options, char **condition, struct passwd *pw,
                }
                arg = criteria = NULL;
                this_result = 1;
-               if ((negate = attrib[0] == '!'))
+               if ((negate = (attrib[0] == '!')))
                        attrib++;
                /* Criterion "all" has no argument and must appear alone */
                if (strcasecmp(attrib, "all") == 0) {

diff --git a/usr.bin/ssh/ssh-agent.c b/usr.bin/ssh/ssh-agent.c
index 0455433..9627e0c 100644 (file)
--- a/usr.bin/ssh/ssh-agent.c
+++ b/usr.bin/ssh/ssh-agent.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-agent.c,v 1.296 2023/03/08 04:43:12 guenther Exp $ */
+/* $OpenBSD: ssh-agent.c,v 1.297 2023/03/09 21:06:24 jcs Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -1010,8 +1010,8 @@ parse_dest_constraint(struct sshbuf *m, struct dest_constraint *dc)
                error_fr(r, "parse");
                goto out;
        }
-       if ((r = parse_dest_constraint_hop(frombuf, &dc->from) != 0) ||
-           (r = parse_dest_constraint_hop(tobuf, &dc->to) != 0))
+       if ((r = parse_dest_constraint_hop(frombuf, &dc->from)) != 0 ||
+           (r = parse_dest_constraint_hop(tobuf, &dc->to)) != 0)
                goto out; /* already logged */
        if (elen != 0) {
                error_f("unsupported extensions (len %zu)", elen);