DDon't call vhangup() on Ultrix until after session. This actually turns out to...

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@6495 dc483132-0cff-0310-8789-dd5450dbe970

diff --git a/src/util/pty/ChangeLog b/src/util/pty/ChangeLog
index a924884aebe2b1f02f78f370fb9d03398582435c..cb95266bda96cf729718dc9a2f128aeaf9cde1a0 100644 (file)
--- a/src/util/pty/ChangeLog
+++ b/src/util/pty/ChangeLog
@@ -1,5 +1,10 @@
 Wed Aug  9 00:16:40 1995  Sam Hartman  <hartmans@tertius.mit.edu>
 
+       * pty-int.h (VHANG_first): Change defines so VHANG_FIRST doesn't
+        get defined under Ultrix because Ultrix can only establish
+        controlling terminal once per process and we need to get
+        controlling terminal again after vhangup().
+
        * getpty.c (pty_getpty): Use the right test for slave buffer length.
 
 Tue Aug  8 22:20:33 1995  Tom Yu  <tlyu@lothlorien.MIT.EDU>

diff --git a/src/util/pty/configure.in b/src/util/pty/configure.in
index 9282a99a46fffc38c45bcafbb0be8735f3c39619..cdb8f55ccfdeb5e3aa3ec4365293bcf6b2a07d0d 100644 (file)
--- a/src/util/pty/configure.in
+++ b/src/util/pty/configure.in
@@ -14,6 +14,10 @@ dnl Make our operating system-specific security checks and definitions for
 dnl login.
 dnl
 case $krb5_cv_host in
+*-*-ultrix*)
+AC_DEFINE([OPEN_CTTY_ONLY_ONCE],[1])
+;;
+
 *-*-aix3*) # AIX has streams include files but not streams TTY
 # Moreover, strops.h trashes sys/ioctl.h
 krb5_cv_has_streams=no

diff --git a/src/util/pty/pty-int.h b/src/util/pty/pty-int.h
index 2bd450c61b008396ef8e829faee2c4617982ac1f..8d5d1aeb8324998114c548e173cb3646d15bd9b8 100644 (file)
--- a/src/util/pty/pty-int.h
+++ b/src/util/pty/pty-int.h
@@ -84,8 +84,8 @@
 #endif
 #endif
 
-#if defined(HAVE_VHANGUP)
-#define VHANG_first /* may not work under Ultrix*/
+#if defined(HAVE_VHANGUP) && !defined(OPEN_CTTY_ONLY_ONCE) 
+#define VHANG_first /* Breaks under Ultrix and others where you cannot get controlling terminal twice.*/
 #define VHANG_LAST
 #endif