- (stevesk) pty.c: HP-UX 10 and 11 don't define TIOCSCTTY. Add error() to

    failed ioctl(TIOCSCTTY) call.

2 files changed, 4 insertions, 6 deletions

diff --git a/ChangeLog b/ChangeLog
index bccc56508..5eb830c21 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,8 @@
  - (bal) SCO Patch to add needed libraries for configure.in.  Patch by
    Phillips Porch <root@theporch.com>
  - (bal) IRIX patch to adding Job Limits.  Patch by Denis Parker <dcp@sgi.com>
+ - (stevesk) pty.c: HP-UX 10 and 11 don't define TIOCSCTTY.  Add error() to
+   failed ioctl(TIOCSCTTY) call.
 
 20001111
  - (djm) Added /etc/primes for kex DH group neg, fixup Makefile.in and
diff --git a/pty.c b/pty.c
index cdf1d1281..40bfd5290 100644
--- a/pty.c
+++ b/pty.c
@@ -238,12 +238,8 @@ pty_make_controlling_tty(int *ttyfd, const char *ttyname)
         /* Make it our controlling tty. */
 #ifdef TIOCSCTTY
         debug("Setting controlling tty using TIOCSCTTY.");
-        /*
-         * We ignore errors from this, because HPSUX defines TIOCSCTTY, but
-         * returns EINVAL with these arguments, and there is absolutely no
-         * documentation.
-         */
-        ioctl(*ttyfd, TIOCSCTTY, NULL);
+        if (ioctl(*ttyfd, TIOCSCTTY, NULL) < 0)
+                error("ioctl(TIOCSCTTY): %.100s", strerror(errno));
 #endif /* TIOCSCTTY */
 #ifdef HAVE_NEWS4
         if (setpgrp(0,0) < 0)