1 files changed, 120 insertions, 35 deletions

diff --git a/packet.c b/packet.c
index 753abebd8..8abd43eb4 100644
--- a/packet.c
+++ b/packet.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: packet.c,v 1.148 2007/06/07 19:37:34 pvalchev Exp $ */
+/* $OpenBSD: packet.c,v 1.157 2008/07/10 18:08:11 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -93,8 +93,6 @@
 static int connection_in = -1;
 static int connection_out = -1;
 
-static int setup_timeout = -1;
-
 /* Protocol flags for the remote side. */
 static u_int remote_protocol_flags = 0;
 
@@ -138,12 +136,18 @@ static int server_side = 0;
 /* Set to true if we are authenticated. */
 static int after_authentication = 0;
 
+int keep_alive_timeouts = 0;
+
+/* Set to the maximum time that we will wait to send or receive a packet */
+static int packet_timeout_ms = -1;
+
 /* Session key information for Encryption and MAC */
 Newkeys *newkeys[MODE_MAX];
 static struct packet_state {
         u_int32_t seqnr;
         u_int32_t packets;
         u_int64_t blocks;
+        u_int64_t bytes;
 } p_read, p_send;
 
 static u_int64_t max_blocks_in, max_blocks_out;
@@ -168,7 +172,7 @@ TAILQ_HEAD(, packet) outgoing;
  * packet_set_encryption_key is called.
  */
 void
-packet_set_connection(int fd_in, int fd_out, int new_setup_timeout)
+packet_set_connection(int fd_in, int fd_out)
 {
         Cipher *none = cipher_by_name("none");
 
@@ -176,7 +180,6 @@ packet_set_connection(int fd_in, int fd_out, int new_setup_timeout)
                 fatal("packet_set_connection: cannot load cipher 'none'");
         connection_in = fd_in;
         connection_out = fd_out;
-        setup_timeout = new_setup_timeout;
         cipher_init(&send_context, none, (const u_char *)"",
             0, NULL, 0, CIPHER_ENCRYPT);
         cipher_init(&receive_context, none, (const u_char *)"",
@@ -189,9 +192,23 @@ packet_set_connection(int fd_in, int fd_out, int new_setup_timeout)
                 buffer_init(&outgoing_packet);
                 buffer_init(&incoming_packet);
                 TAILQ_INIT(&outgoing);
+                p_send.packets = p_read.packets = 0;
         }
 }
 
+void
+packet_set_timeout(int timeout, int count)
+{
+        if (timeout == 0 || count == 0) {
+                packet_timeout_ms = -1;
+                return;
+        }
+        if ((INT_MAX / 1000) / count < timeout)
+                packet_timeout_ms = INT_MAX;
+        else
+                packet_timeout_ms = timeout * count * 1000;
+}
+
 /* Returns 1 if remote host is connected via socket, 0 if not. */
 
 int
@@ -296,18 +313,25 @@ packet_get_ssh1_cipher(void)
 }
 
 void
-packet_get_state(int mode, u_int32_t *seqnr, u_int64_t *blocks, u_int32_t *packets)
+packet_get_state(int mode, u_int32_t *seqnr, u_int64_t *blocks, u_int32_t *packets,
+    u_int64_t *bytes)
 {
         struct packet_state *state;
 
         state = (mode == MODE_IN) ? &p_read : &p_send;
-        *seqnr = state->seqnr;
-        *blocks = state->blocks;
-        *packets = state->packets;
+        if (seqnr)
+                *seqnr = state->seqnr;
+        if (blocks)
+                *blocks = state->blocks;
+        if (packets)
+                *packets = state->packets;
+        if (bytes)
+                *bytes = state->bytes;
 }
 
 void
-packet_set_state(int mode, u_int32_t seqnr, u_int64_t blocks, u_int32_t packets)
+packet_set_state(int mode, u_int32_t seqnr, u_int64_t blocks, u_int32_t packets,
+    u_int64_t bytes)
 {
         struct packet_state *state;
 
@@ -315,6 +339,7 @@ packet_set_state(int mode, u_int32_t seqnr, u_int64_t blocks, u_int32_t packets)
         state->seqnr = seqnr;
         state->blocks = blocks;
         state->packets = packets;
+        state->bytes = bytes;
 }
 
 /* returns 1 if connection is via ipv4 */
@@ -593,7 +618,8 @@ packet_send1(void)
         fprintf(stderr, "encrypted: ");
         buffer_dump(&output);
 #endif
-
+        p_send.packets++;
+        p_send.bytes += len + buffer_len(&outgoing_packet);
         buffer_clear(&outgoing_packet);
 
         /*
@@ -819,6 +845,7 @@ packet_send2_wrapped(void)
                 if (!(datafellows & SSH_BUG_NOREKEY))
                         fatal("XXX too many packets with same key");
         p_send.blocks += (packet_length + 4) / block_size;
+        p_send.bytes += packet_length + 4;
         buffer_clear(&outgoing_packet);
 
         if (type == SSH2_MSG_NEWKEYS)
@@ -892,10 +919,11 @@ packet_send(void)
 int
 packet_read_seqnr(u_int32_t *seqnr_p)
 {
-        int type, len;
+        int type, len, ret, ms_remain;
         fd_set *setp;
         char buf[8192];
-        struct timeval tv, *tvp;
+        struct timeval timeout, start, *timeoutp = NULL;
+
         DBG(debug("packet_read()"));
 
         setp = (fd_set *)xcalloc(howmany(connection_in+1, NFDBITS),
@@ -927,21 +955,35 @@ packet_read_seqnr(u_int32_t *seqnr_p)
                     sizeof(fd_mask));
                 FD_SET(connection_in, setp);
 
-                if (setup_timeout > 0) {
-                        tvp = &tv;
-                        tv.tv_sec = setup_timeout;
-                        tv.tv_usec = 0;
-                } else
-                        tvp = NULL;
-
+                if (packet_timeout_ms > 0) {
+                        ms_remain = packet_timeout_ms;
+                        timeoutp = &timeout;
+                }
                 /* Wait for some data to arrive. */
-                while (select(connection_in + 1, setp, NULL, NULL, tvp) == -1 &&
-                    (errno == EAGAIN || errno == EINTR))
-                        ;
-
-                if (!FD_ISSET(connection_in, setp))
-                        fatal("packet_read: Setup timeout expired, giving up");
-
+                for (;;) {
+                        if (packet_timeout_ms != -1) {
+                                ms_to_timeval(&timeout, ms_remain);
+                                gettimeofday(&start, NULL);
+                        }
+                        if ((ret = select(connection_in + 1, setp, NULL,
+                            NULL, timeoutp)) >= 0)
+                                break;
+                        if (errno != EAGAIN && errno != EINTR &&
+                            errno != EWOULDBLOCK)
+                                break;
+                        if (packet_timeout_ms == -1)
+                                continue;
+                        ms_subtract_diff(&start, &ms_remain);
+                        if (ms_remain <= 0) {
+                                ret = 0;
+                                break;
+                        }
+                }
+                if (ret == 0) {
+                        logit("Connection to %.200s timed out while "
+                            "waiting to read", get_remote_ipaddr());
+                        cleanup_exit(255);
+                }
                 /* Read data from the socket. */
                 len = read(connection_in, buf, sizeof(buf));
                 if (len == 0) {
@@ -1066,6 +1108,8 @@ packet_read_poll1(void)
                 buffer_append(&incoming_packet, buffer_ptr(&compression_buffer),
                     buffer_len(&compression_buffer));
         }
+        p_read.packets++;
+        p_read.bytes += padded_len + 4;
         type = buffer_get_char(&incoming_packet);
         if (type < SSH_MSG_MIN || type > SSH_MSG_MAX)
                 packet_disconnect("Invalid ssh1 packet type: %d", type);
@@ -1154,6 +1198,7 @@ packet_read_poll2(u_int32_t *seqnr_p)
                 if (!(datafellows & SSH_BUG_NOREKEY))
                         fatal("XXX too many packets with same key");
         p_read.blocks += (packet_length + 4) / block_size;
+        p_read.bytes += packet_length + 4;
 
         /* get padlen */
         cp = buffer_ptr(&incoming_packet);
@@ -1206,10 +1251,13 @@ packet_read_poll_seqnr(u_int32_t *seqnr_p)
         for (;;) {
                 if (compat20) {
                         type = packet_read_poll2(seqnr_p);
-                        if (type)
+                        if (type) {
+                                keep_alive_timeouts = 0;
                                 DBG(debug("received packet type %d", type));
+                        }
                         switch (type) {
                         case SSH2_MSG_IGNORE:
+                                debug3("Received SSH2_MSG_IGNORE");
                                 break;
                         case SSH2_MSG_DEBUG:
                                 packet_get_char();
@@ -1342,6 +1390,12 @@ packet_get_string(u_int *length_ptr)
         return buffer_get_string(&incoming_packet, length_ptr);
 }
 
+void *
+packet_get_string_ptr(u_int *length_ptr)
+{
+        return buffer_get_string_ptr(&incoming_packet, length_ptr);
+}
+
 /*
  * Sends a diagnostic message from the server to the client.  This message
  * can be sent at any time (but not while constructing another message). The
@@ -1436,16 +1490,19 @@ packet_write_poll(void)
 
         if (len > 0) {
                 len = write(connection_out, buffer_ptr(&output), len);
-                if (len <= 0) {
-                        if (errno == EAGAIN)
+                if (len == -1) {
+                        if (errno == EINTR || errno == EAGAIN ||
+                            errno == EWOULDBLOCK)
                                 return;
-                        else
-                                fatal("Write failed: %.100s", strerror(errno));
+                        fatal("Write failed: %.100s", strerror(errno));
                 }
+                if (len == 0)
+                        fatal("Write connection closed");
                 buffer_consume(&output, len);
         }
 }
 
+
 /*
  * Calls packet_write_poll repeatedly until all pending output data has been
  * written.
@@ -1455,6 +1512,8 @@ void
 packet_write_wait(void)
 {
         fd_set *setp;
+        int ret, ms_remain;
+        struct timeval start, timeout, *timeoutp = NULL;
 
         setp = (fd_set *)xcalloc(howmany(connection_out + 1, NFDBITS),
             sizeof(fd_mask));
@@ -1463,9 +1522,35 @@ packet_write_wait(void)
                 memset(setp, 0, howmany(connection_out + 1, NFDBITS) *
                     sizeof(fd_mask));
                 FD_SET(connection_out, setp);
-                while (select(connection_out + 1, NULL, setp, NULL, NULL) == -1 &&
-                    (errno == EAGAIN || errno == EINTR))
-                        ;
+
+                if (packet_timeout_ms > 0) {
+                        ms_remain = packet_timeout_ms;
+                        timeoutp = &timeout;
+                }
+                for (;;) {
+                        if (packet_timeout_ms != -1) {
+                                ms_to_timeval(&timeout, ms_remain);
+                                gettimeofday(&start, NULL);
+                        }
+                        if ((ret = select(connection_out + 1, NULL, setp,
+                            NULL, timeoutp)) >= 0)
+                                break;
+                        if (errno != EAGAIN && errno != EINTR &&
+                            errno != EWOULDBLOCK)
+                                break;
+                        if (packet_timeout_ms == -1)
+                                continue;
+                        ms_subtract_diff(&start, &ms_remain);
+                        if (ms_remain <= 0) {
+                                ret = 0;
+                                break;
+                        }
+                }
+                if (ret == 0) {
+                        logit("Connection to %.200s timed out while "
+                            "waiting to write", get_remote_ipaddr());
+                        cleanup_exit(255);
+                }
                 packet_write_poll();
         }
         xfree(setp);