Bug 312

Summary:	canhost.h needs to be included
Product:	Portable OpenSSH	Reporter:	Dirk Meyer <dirk.meyer>
Component:	Build system	Assignee:	OpenSSH Bugzilla mailing list <openssh-bugs>
Status:	CLOSED FIXED
Severity:	normal
Priority:	P2
Version:	-current
Hardware:	All
OS:	FreeBSD

Description Dirk Meyer 2002-06-28 15:45:11 AEST

--- auth1.c.orig        Fri Jun 21 08:21:11 2002
+++ auth1.c     Fri Jun 28 06:57:42 2002
@@ -26,6 +26,7 @@
 #include "session.h"
 #include "uidswap.h"
 #include "monitor_wrap.h"
+#include "canohost.h"

 /* import */
 extern ServerOptions options;
--- auth2.c.orig        Fri Jun 21 08:21:11 2002
+++ auth2.c     Fri Jun 28 06:57:56 2002
@@ -35,6 +35,7 @@
 #include "dispatch.h"
 #include "pathnames.h"
 #include "monitor_wrap.h"
+#include "canohost.h"

 /* import */
 extern ServerOptions options;

Comment 1 Kevin Steves 2002-07-18 14:07:45 AEST

why is it required?
i don't see any canohost.h functions in those files.

Comment 2 Dirk Meyer 2002-07-18 16:11:02 AEST

Well it is conditional.

# grep -C -n get_canonical_hostname auth2.c.orig 
236-                    /* XXX: privsep */
237-                    loginfailed(authctxt->user,
238:                        get_canonical_hostname(options.verify_reverse_mapping),
239-                        "ssh");
240-#endif /* WITH_AIXAUTHENTICATE */

# grep -C -n get_canonical_hostname auth1.c.orig 
327-                    /* XXX: privsep */
328-                    loginfailed(authctxt->user,
329:                        get_canonical_hostname(options.verify_reverse_mapping),
330-                        "ssh");
331-#endif /* WITH_AIXAUTHENTICATE */

Comment 3 Kevin Steves 2002-07-19 10:06:40 AEST

i see.
they were re/moved about 2 weeks ago.

Comment 4 Damien Miller 2004-04-14 12:24:18 AEST

Mass change of RESOLVED bugs to CLOSED