remove now obsolete ctx from ssh_dispatch_run; ok djm@

diff --git a/usr.bin/ssh/auth2.c b/usr.bin/ssh/auth2.c
index fcdfbdb..f83131a 100644 (file)
--- a/usr.bin/ssh/auth2.c
+++ b/usr.bin/ssh/auth2.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: auth2.c,v 1.141 2017/05/31 05:34:14 markus Exp $ */
+/* $OpenBSD: auth2.c,v 1.142 2017/05/31 07:00:13 markus Exp $ */
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
  *
@@ -159,7 +159,7 @@ do_authentication2(Authctxt *authctxt)
        ssh->authctxt = authctxt;               /* XXX move to caller */
        ssh_dispatch_init(ssh, &dispatch_protocol_error);
        ssh_dispatch_set(ssh, SSH2_MSG_SERVICE_REQUEST, &input_service_request);
-       ssh_dispatch_run_fatal(ssh, DISPATCH_BLOCK, &authctxt->success, ssh);
+       ssh_dispatch_run_fatal(ssh, DISPATCH_BLOCK, &authctxt->success);
        ssh->authctxt = NULL;
 }
 

diff --git a/usr.bin/ssh/clientloop.c b/usr.bin/ssh/clientloop.c
index 400e8bb..c0c1b35 100644 (file)
--- a/usr.bin/ssh/clientloop.c
+++ b/usr.bin/ssh/clientloop.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: clientloop.c,v 1.297 2017/05/30 14:23:52 markus Exp $ */
+/* $OpenBSD: clientloop.c,v 1.298 2017/05/31 07:00:13 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -1158,7 +1158,7 @@ process_escapes(Channel *c, Buffer *bin, Buffer *bout, Buffer *berr,
 static void
 client_process_buffered_input_packets(void)
 {
-       dispatch_run(DISPATCH_NONBLOCK, &quit_pending, active_state);
+       ssh_dispatch_run_fatal(active_state, DISPATCH_NONBLOCK, &quit_pending);
 }
 
 /* scan buf[] for '~' before sending data to the peer */

diff --git a/usr.bin/ssh/dispatch.c b/usr.bin/ssh/dispatch.c
index 0436dc0..7bb9342 100644 (file)
--- a/usr.bin/ssh/dispatch.c
+++ b/usr.bin/ssh/dispatch.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: dispatch.c,v 1.30 2017/05/30 14:23:52 markus Exp $ */
+/* $OpenBSD: dispatch.c,v 1.31 2017/05/31 07:00:13 markus Exp $ */
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
  *
@@ -83,8 +83,7 @@ ssh_dispatch_set(struct ssh *ssh, int type, dispatch_fn *fn)
 }
 
 int
-ssh_dispatch_run(struct ssh *ssh, int mode, volatile sig_atomic_t *done,
-    void *ctxt)
+ssh_dispatch_run(struct ssh *ssh, int mode, volatile sig_atomic_t *done)
 {
        int r;
        u_char type;
@@ -109,8 +108,7 @@ ssh_dispatch_run(struct ssh *ssh, int mode, volatile sig_atomic_t *done,
                                ssh->dispatch_skip_packets--;
                                continue;
                        }
-                       /* XXX 'ssh' will replace 'ctxt' later */
-                       r = (*ssh->dispatch[type])(type, seqnr, ctxt);
+                       r = (*ssh->dispatch[type])(type, seqnr, ssh);
                        if (r != 0)
                                return r;
                } else {
@@ -126,11 +124,10 @@ ssh_dispatch_run(struct ssh *ssh, int mode, volatile sig_atomic_t *done,
 }
 
 void
-ssh_dispatch_run_fatal(struct ssh *ssh, int mode, volatile sig_atomic_t *done,
-    void *ctxt)
+ssh_dispatch_run_fatal(struct ssh *ssh, int mode, volatile sig_atomic_t *done)
 {
        int r;
 
-       if ((r = ssh_dispatch_run(ssh, mode, done, ctxt)) != 0)
+       if ((r = ssh_dispatch_run(ssh, mode, done)) != 0)
                sshpkt_fatal(ssh, __func__, r);
 }

diff --git a/usr.bin/ssh/dispatch.h b/usr.bin/ssh/dispatch.h
index 7dfc74e..17a6f3d 100644 (file)
--- a/usr.bin/ssh/dispatch.h
+++ b/usr.bin/ssh/dispatch.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: dispatch.h,v 1.13 2017/05/30 14:23:52 markus Exp $ */
+/* $OpenBSD: dispatch.h,v 1.14 2017/05/31 07:00:13 markus Exp $ */
 
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
@@ -43,8 +43,8 @@ int   dispatch_protocol_ignore(int, u_int32_t, struct ssh *);
 void   ssh_dispatch_init(struct ssh *, dispatch_fn *);
 void   ssh_dispatch_set(struct ssh *, int, dispatch_fn *);
 void   ssh_dispatch_range(struct ssh *, u_int, u_int, dispatch_fn *);
-int    ssh_dispatch_run(struct ssh *, int, volatile sig_atomic_t *, void *);
-void   ssh_dispatch_run_fatal(struct ssh *, int, volatile sig_atomic_t *, void *);
+int    ssh_dispatch_run(struct ssh *, int, volatile sig_atomic_t *);
+void   ssh_dispatch_run_fatal(struct ssh *, int, volatile sig_atomic_t *);
 
 #define dispatch_init(dflt) \
        ssh_dispatch_init(active_state, (dflt))
@@ -52,7 +52,5 @@ void  ssh_dispatch_run_fatal(struct ssh *, int, volatile sig_atomic_t *, void *);
        ssh_dispatch_range(active_state, (from), (to), (fn))
 #define dispatch_set(type, fn) \
        ssh_dispatch_set(active_state, (type), (fn))
-#define dispatch_run(mode, done, ctxt) \
-       ssh_dispatch_run_fatal(active_state, (mode), (done), (ctxt))
 
 #endif

diff --git a/usr.bin/ssh/serverloop.c b/usr.bin/ssh/serverloop.c
index f02b837..f7458f0 100644 (file)
--- a/usr.bin/ssh/serverloop.c
+++ b/usr.bin/ssh/serverloop.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: serverloop.c,v 1.192 2017/05/30 14:23:52 markus Exp $ */
+/* $OpenBSD: serverloop.c,v 1.193 2017/05/31 07:00:13 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -316,7 +316,7 @@ process_output(fd_set *writeset, int connection_out)
 static void
 process_buffered_input_packets(void)
 {
-       dispatch_run(DISPATCH_NONBLOCK, NULL, active_state);
+       ssh_dispatch_run_fatal(active_state, DISPATCH_NONBLOCK, NULL);
 }
 
 static void

diff --git a/usr.bin/ssh/ssh-keyscan.c b/usr.bin/ssh/ssh-keyscan.c
index 8dba83c..c8237e8 100644 (file)
--- a/usr.bin/ssh/ssh-keyscan.c
+++ b/usr.bin/ssh/ssh-keyscan.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-keyscan.c,v 1.113 2017/04/30 23:28:42 djm Exp $ */
+/* $OpenBSD: ssh-keyscan.c,v 1.114 2017/05/31 07:00:13 markus Exp $ */
 /*
  * Copyright 1995, 1996 by David Mazieres <dm@lcs.mit.edu>.
  *
@@ -251,7 +251,7 @@ keygrab_ssh2(con *c)
         * do the key-exchange until an error occurs or until
         * the key_print_wrapper() callback sets c_done.
         */
-       ssh_dispatch_run(c->c_ssh, DISPATCH_BLOCK, &c->c_done, c->c_ssh);
+       ssh_dispatch_run(c->c_ssh, DISPATCH_BLOCK, &c->c_done);
 }
 
 static void

diff --git a/usr.bin/ssh/sshconnect2.c b/usr.bin/ssh/sshconnect2.c
index be0a4e6..3a5833e 100644 (file)
--- a/usr.bin/ssh/sshconnect2.c
+++ b/usr.bin/ssh/sshconnect2.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sshconnect2.c,v 1.262 2017/05/31 05:08:46 djm Exp $ */
+/* $OpenBSD: sshconnect2.c,v 1.263 2017/05/31 07:00:13 markus Exp $ */
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
  * Copyright (c) 2008 Damien Miller.  All rights reserved.
@@ -209,7 +209,7 @@ ssh_kex2(char *host, struct sockaddr *hostaddr, u_short port)
        kex->server_version_string=server_version_string;
        kex->verify_host_key=&verify_host_key_callback;
 
-       dispatch_run(DISPATCH_BLOCK, &kex->done, active_state);
+       ssh_dispatch_run_fatal(active_state, DISPATCH_BLOCK, &kex->done);
 
        /* remove ext-info from the KEX proposals for rekeying */
        myproposal[PROPOSAL_KEX_ALGS] =
@@ -393,7 +393,7 @@ ssh_userauth2(const char *local_user, const char *server_user, char *host,
        ssh_dispatch_init(ssh, &input_userauth_error);
        ssh_dispatch_set(ssh, SSH2_MSG_EXT_INFO, &input_userauth_ext_info);
        ssh_dispatch_set(ssh, SSH2_MSG_SERVICE_ACCEPT, &input_userauth_service_accept);
-       ssh_dispatch_run(ssh, DISPATCH_BLOCK, &authctxt.success, ssh);  /* loop until success */
+       ssh_dispatch_run_fatal(ssh, DISPATCH_BLOCK, &authctxt.success); /* loop until success */
        ssh->authctxt = NULL;
 
        pubkey_cleanup(&authctxt);

diff --git a/usr.bin/ssh/sshd.c b/usr.bin/ssh/sshd.c
index 9044016..a05549b 100644 (file)
--- a/usr.bin/ssh/sshd.c
+++ b/usr.bin/ssh/sshd.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: sshd.c,v 1.488 2017/05/30 08:52:20 markus Exp $ */
+/* $OpenBSD: sshd.c,v 1.489 2017/05/31 07:00:13 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -2006,7 +2006,7 @@ do_ssh2_kex(void)
        kex->host_key_index=&get_hostkey_index;
        kex->sign = sshd_hostkey_sign;
 
-       dispatch_run(DISPATCH_BLOCK, &kex->done, active_state);
+       ssh_dispatch_run_fatal(active_state, DISPATCH_BLOCK, &kex->done);
 
        session_id2 = kex->session_id;
        session_id2_len = kex->session_id_len;