IP options

Scott Neugroschl scott_n at xypro.com
Fri Aug 22 09:51:03 EST 2008


>From: Scott Neugroschl 
>
>In addition, there's an error in canohost.c/getremotehostname(), that
the fix
>for 1179 introduced (to wit, the variable ntop is uninitialized at the
point
>of call to check_ip_options).  I've got to run off to a meeting, but
I'll
>try to post a patch later today.

Here's the proposed patch:


Index: canohost.c
===================================================================
RCS file: /cvs/openssh/canohost.c,v
retrieving revision 1.73
diff -u -r1.73 canohost.c
--- canohost.c	12 Jun 2008 18:46:45 -0000	1.73
+++ canohost.c	21 Aug 2008 23:45:32 -0000
@@ -58,8 +58,13 @@
 		cleanup_exit(255);
 	}
 
-	if (from.ss_family == AF_INET)
+	if (from.ss_family == AF_INET) {
+		if (getnameinfo((struct sockaddr *)&from, fromlen, ntop,
sizeof(ntop),
+		    NULL, 0, NI_NUMERICHOST) != 0)
+			fatal("get_remote_hostname: getnameinfo
NI_NUMERICHOST failed");
+
 		check_ip_options(sock, ntop);
+	}
 
 	ipv64_normalise_mapped(&from, &fromlen);



More information about the openssh-unix-dev mailing list