# HG changeset patch
# User Matt Johnston <matt@ucc.asn.au>
# Date 1170585179 0
# Node ID b895f91c2ee662dd58634c1d8b61a2ad61ca623f
# Parent  a707e6148060f706d2b42d3604708902b8924f34# Parent  0e69e948cabab579d365bada7cb88fc0c416ed3e
merge of 'b1dd3b94e60a07a176dba2b035ac79968595990a'
     and 'bcb33fce2fad01a7626598209d43af3571bd86f0'

diff -r a707e6148060 -r b895f91c2ee6 cli-authpubkey.c
--- a/cli-authpubkey.c	Sat Feb 03 08:20:34 2007 +0000
+++ b/cli-authpubkey.c	Sun Feb 04 10:32:59 2007 +0000
@@ -60,8 +60,8 @@
 
 void recv_msg_userauth_pk_ok() {
 
-	struct SignKeyList *keyitem;
-	buffer* keybuf;
+	struct SignKeyList *keyitem = NULL;
+	buffer* keybuf = NULL;
 	char* algotype = NULL;
 	unsigned int algolen;
 	int keytype;
@@ -122,6 +122,8 @@
 	} else {
 		TRACE(("That was whacky. We got told that a key was valid, but it didn't match our list. Sounds like dodgy code on Dropbear's part"))
 	}
+	
+	buf_free(keybuf);
 
 	TRACE(("leave recv_msg_userauth_pk_ok"))
 }
diff -r a707e6148060 -r b895f91c2ee6 cli-runopts.c
--- a/cli-runopts.c	Sat Feb 03 08:20:34 2007 +0000
+++ b/cli-runopts.c	Sun Feb 04 10:32:59 2007 +0000
@@ -373,8 +373,7 @@
 		TRACE(("connectaddr == NULL"))
 		goto fail;
 	}
-
-	connectaddr[0] = '\0';
+	*connectaddr = '\0';
 	connectaddr++;
 
 	connectport = strchr(connectaddr, ':');
@@ -382,8 +381,7 @@
 		TRACE(("connectport == NULL"))
 		goto fail;
 	}
-
-	connectport[0] = '\0';
+	*connectport = '\0';
 	connectport++;
 
 	newfwd = (struct TCPFwdList*)m_malloc(sizeof(struct TCPFwdList));
@@ -417,6 +415,8 @@
 	newfwd->next = *fwdlist;
 	*fwdlist = newfwd;
 
+	m_free(str);
+
 	TRACE(("leave addforward: done"))
 	return;
 
diff -r a707e6148060 -r b895f91c2ee6 common-algo.c
--- a/common-algo.c	Sat Feb 03 08:20:34 2007 +0000
+++ b/common-algo.c	Sun Feb 04 10:32:59 2007 +0000
@@ -31,6 +31,8 @@
 
 /* Mappings for ciphers, parameters are
    {&cipher_desc, keysize, blocksize} */
+/* NOTE: if keysize > 2*SHA1_HASH_SIZE, code such as hashkeys()
+   needs revisiting */
 
 #ifdef DROPBEAR_AES256_CBC
 static const struct dropbear_cipher dropbear_aes256 = 
diff -r a707e6148060 -r b895f91c2ee6 common-kex.c
--- a/common-kex.c	Sat Feb 03 08:20:34 2007 +0000
+++ b/common-kex.c	Sun Feb 04 10:32:59 2007 +0000
@@ -217,12 +217,10 @@
  * already initialised hash_state hs, which should already have processed
  * the dh_K and hash, since these are common. X is the letter 'A', 'B' etc.
  * out must have at least min(SHA1_HASH_SIZE, outlen) bytes allocated.
- * The output will only be expanded once, since that is all that is required
- * (for 3DES and SHA, with 24 and 20 bytes respectively). 
+ * The output will only be expanded once, as we are assured that
+ * outlen <= 2*SHA1_HASH_SIZE for all known hashes.
  *
- * See Section 5.2 of the IETF secsh Transport Draft for details */
-
-/* Duplicated verbatim from kex.c --mihnea */
+ * See Section 7.2 of rfc4253 (ssh transport) for details */
 static void hashkeys(unsigned char *out, int outlen, 
 		const hash_state * hs, const unsigned char X) {
 
diff -r a707e6148060 -r b895f91c2ee6 dbutil.c
--- a/dbutil.c	Sat Feb 03 08:20:34 2007 +0000
+++ b/dbutil.c	Sun Feb 04 10:32:59 2007 +0000
@@ -286,9 +286,9 @@
 			len = 20 + strlen(strerror(err));
 			*errstring = (char*)m_malloc(len);
 			snprintf(*errstring, len, "Error listening: %s", strerror(err));
-			TRACE(("leave dropbear_listen: failure, %s", strerror(err)))
-			return -1;
 		}
+		TRACE(("leave dropbear_listen: failure, %s", strerror(err)))
+		return -1;
 	}
 
 	TRACE(("leave dropbear_listen: success, %d socks bound", nsock))
diff -r a707e6148060 -r b895f91c2ee6 random.c
--- a/random.c	Sat Feb 03 08:20:34 2007 +0000
+++ b/random.c	Sun Feb 04 10:32:59 2007 +0000
@@ -234,8 +234,7 @@
 
 		/* keep regenerating until we get one satisfying
 		 * 0 < rand < max    */
-	} while ( ( (max != NULL) && (mp_cmp(rand, max) != MP_LT) )
-			|| (mp_cmp_d(rand, 0) != MP_GT) );
+	} while (mp_cmp(rand, max) != MP_LT);
 	m_burn(randbuf, len);
 	m_free(randbuf);
 }
diff -r a707e6148060 -r b895f91c2ee6 svr-session.c
--- a/svr-session.c	Sat Feb 03 08:20:34 2007 +0000
+++ b/svr-session.c	Sun Feb 04 10:32:59 2007 +0000
@@ -181,10 +181,15 @@
 
 	if (!svr_opts.usingsyslog || havetrace)
 	{
+		struct tm * local_tm = NULL;
 		timesec = time(NULL);
-		if (strftime(datestr, sizeof(datestr), "%b %d %H:%M:%S", 
-					localtime(&timesec)) == 0) {
-			datestr[0] = '?'; datestr[1] = '\0';
+		local_tm = localtime(&timesec);
+		if (local_tm == NULL
+			|| strftime(datestr, sizeof(datestr), "%b %d %H:%M:%S", 
+						localtime(&timesec)) == 0)
+		{
+			// upon failure, just print the epoch-seconds time.
+			snprintf(datestr, sizeof(datestr), "%d", timesec);
 		}
 		fprintf(stderr, "[%d] %s %s\n", getpid(), datestr, printbuf);
 	}
diff -r a707e6148060 -r b895f91c2ee6 svr-tcpfwd.c