treat ssh_packet_write_wait() errors as fatal; ok djm@
authormarkus <markus@openbsd.org>
Wed, 11 Jul 2018 18:55:11 +0000 (18:55 +0000)
committermarkus <markus@openbsd.org>
Wed, 11 Jul 2018 18:55:11 +0000 (18:55 +0000)
usr.bin/ssh/auth2-chall.c
usr.bin/ssh/auth2-pubkey.c
usr.bin/ssh/sshconnect2.c

index 9e91ee2..3941726 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: auth2-chall.c,v 1.49 2018/07/09 21:35:50 markus Exp $ */
+/* $OpenBSD: auth2-chall.c,v 1.50 2018/07/11 18:55:11 markus Exp $ */
 /*
  * Copyright (c) 2001 Markus Friedl.  All rights reserved.
  * Copyright (c) 2001 Per Allansson.  All rights reserved.
@@ -234,9 +234,9 @@ send_userauth_info_request(struct ssh *ssh)
                    (r = sshpkt_put_u8(ssh, echo_on[i])) != 0)
                        fatal("%s: %s", __func__, ssh_err(r));
        }
-       if ((r = sshpkt_send(ssh)) != 0)
+       if ((r = sshpkt_send(ssh)) != 0 ||
+           (r = ssh_packet_write_wait(ssh)) != 0)
                fatal("%s: %s", __func__, ssh_err(r));
-       ssh_packet_write_wait(ssh);
 
        for (i = 0; i < kbdintctxt->nreq; i++)
                free(prompts[i]);
index ad2b668..195da5e 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: auth2-pubkey.c,v 1.81 2018/07/09 21:35:50 markus Exp $ */
+/* $OpenBSD: auth2-pubkey.c,v 1.82 2018/07/11 18:55:11 markus Exp $ */
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
  *
@@ -215,9 +215,9 @@ userauth_pubkey(struct ssh *ssh)
                            != 0 ||
                            (r = sshpkt_put_cstring(ssh, pkalg)) != 0 ||
                            (r = sshpkt_put_string(ssh, pkblob, blen)) != 0 ||
-                           (r = sshpkt_send(ssh)) != 0)
+                           (r = sshpkt_send(ssh)) != 0 ||
+                           (r = ssh_packet_write_wait(ssh)) != 0)
                                fatal("%s: %s", __func__, ssh_err(r));
-                       ssh_packet_write_wait(ssh);
                        authctxt->postponed = 1;
                }
        }
index ddf5700..c7906af 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: sshconnect2.c,v 1.279 2018/07/11 18:53:29 markus Exp $ */
+/* $OpenBSD: sshconnect2.c,v 1.280 2018/07/11 18:55:11 markus Exp $ */
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
  * Copyright (c) 2008 Damien Miller.  All rights reserved.
@@ -226,9 +226,9 @@ ssh_kex2(char *host, struct sockaddr *hostaddr, u_short port)
        /* send 1st encrypted/maced/compressed message */
        if ((r = sshpkt_start(ssh, SSH2_MSG_IGNORE)) != 0 ||
            (r = sshpkt_put_cstring(ssh, "markus")) != 0 ||
-           (r = sshpkt_send(ssh)) != 0)
+           (r = sshpkt_send(ssh)) != 0 ||
+           (r = ssh_packet_write_wait(ssh)) != 0)
                fatal("%s: %s", __func__, ssh_err(r));
-       ssh_packet_write_wait(ssh);
 #endif
 }