From: dtucker <dtucker>
Date: Mon, 3 Oct 2005 08:23:44 +0000 (+0000)
Subject:    - dtucker@cvs.openbsd.org 2005/10/03 07:44:42
X-Git-Tag: V_4_3_P1~211
X-Git-Url: http://andersk.mit.edu/gitweb/openssh.git/commitdiff_plain/d2130e1fa193455d385687eac81ed6d3f5f0fd1d?hp=8f921a4ae7f7541671efdabb6be629dd91f47eb1

   - dtucker@cvs.openbsd.org 2005/10/03 07:44:42
     [canohost.c]
     Relocate check_ip_options call to prevent logging of garbage for
     connections with IP options set.  bz#1092 from David Leonard,
     "looks good" deraadt@
---

diff --git a/ChangeLog b/ChangeLog
index caed18d4..4120ac33 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -39,6 +39,11 @@
    - jaredy@cvs.openbsd.org 2005/09/30 20:34:26
      [ssh-keyscan.1]
      deploy .An -nosplit; ok jmc
+   - dtucker@cvs.openbsd.org 2005/10/03 07:44:42
+     [canohost.c]
+     Relocate check_ip_options call to prevent logging of garbage for
+     connections with IP options set.  bz#1092 from David Leonard,
+     "looks good" deraadt@
 
 20050930
  - (dtucker) [openbsd-compat/openbsd-compat.h] Bug #1096: Add prototype
diff --git a/canohost.c b/canohost.c
index c27086bf..0c4d36ff 100644
--- a/canohost.c
+++ b/canohost.c
@@ -12,7 +12,7 @@
  */
 
 #include "includes.h"
-RCSID("$OpenBSD: canohost.c,v 1.44 2005/06/17 02:44:32 djm Exp $");
+RCSID("$OpenBSD: canohost.c,v 1.45 2005/10/03 07:44:42 dtucker Exp $");
 
 #include "packet.h"
 #include "xmalloc.h"
@@ -43,9 +43,6 @@ get_remote_hostname(int sock, int use_dns)
 		cleanup_exit(255);
 	}
 
-	if (from.ss_family == AF_INET)
-		check_ip_options(sock, ntop);
-
 	ipv64_normalise_mapped(&from, &fromlen);
 
 	if (from.ss_family == AF_INET6)
@@ -55,6 +52,9 @@ get_remote_hostname(int sock, int use_dns)
 	    NULL, 0, NI_NUMERICHOST) != 0)
 		fatal("get_remote_hostname: getnameinfo NI_NUMERICHOST failed");
 
+	if (from.ss_family == AF_INET)
+		check_ip_options(sock, ntop);
+
 	if (!use_dns)
 		return xstrdup(ntop);