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

    failed ioctl(TIOCSCTTY) call.

diff --git a/ChangeLog b/ChangeLog
index bccc5650893b900737166435db24f7e06c0d163e..5eb830c21d34918340ae99c343527781649c58e0 100644 (file)
--- 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 cdf1d1281c526dce579d7e1c688eaba50ef5c2e9..40bfd52906d41566681fedc5d3bda20bbe262973 100644 (file)
--- 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)