bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

ptsname_r on Solaris


From: Bruno Haible
Subject: ptsname_r on Solaris
Date: Sun, 24 Jun 2012 18:46:50 +0200
User-agent: KMail/4.7.4 (Linux/3.1.10-1.9-desktop; KDE/4.7.4; x86_64; ; )

When running the test for module 'ptsname_r' on Solaris *interactively*,
I'm seeing this error:

  test-ptsname_r.c:163: assertion failed
  FAIL: test-ptsname_r

Solaris needs a different approach for ptsname_r. The (not reentrant)
ptsname function from the OS does the following system calls, observed
via 'truss':

  ioctl(3, I_STR, 0xFFBFF830)                     = 0
  fstat64(3, 0xFFBFF798)                          = 0
  access("/dev/pts/8", 0)                         = 0

With similar system calls in gnulib's ptsname_r.c this function works fine.
I'm committing this patch:


2012-06-24  Bruno Haible  <address@hidden>

        ptsname_r: Add support for Solaris.
        * lib/ptsname_r.c (__ptsname_r): Add a different implementation for
        Solaris.

--- lib/ptsname_r.c.orig        Sun Jun 24 18:40:04 2012
+++ lib/ptsname_r.c     Sun Jun 24 18:36:41 2012
@@ -42,6 +42,17 @@
 
 #endif
 
+#ifdef __sun
+/* Get ioctl() and 'struct strioctl'.  */
+# include <stropts.h>
+/* Get ISPTM.  */
+# include <sys/stream.h>
+# include <sys/ptms.h>
+/* Get the major, minor macros.  */
+# include <sys/sysmacros.h>
+# include <stdio.h>
+#endif
+
 
 /* Store at most BUFLEN characters of the pathname of the slave pseudo
    terminal associated with the master FD is open on in BUF.
@@ -59,6 +70,44 @@
       return EINVAL;
     }
 
+#if defined __sun /* Solaris */
+  if (fstat (fd, &st) < 0)
+    return errno;
+  if (!(S_ISCHR (st.st_mode) && major (st.st_rdev) == 0))
+    {
+      errno = ENOTTY;
+      return errno;
+    }
+  {
+    /* Master ptys can be recognized through a STREAMS ioctl.  See
+       "STREAMS-based Pseudo-Terminal Subsystem"
+       <http://docs.oracle.com/cd/E18752_01/html/816-4855/termsub15-44781.html>
+       and "STREAMS ioctl commands"
+       <http://docs.oracle.com/cd/E18752_01/html/816-5177/streamio-7i.html>
+     */
+    struct strioctl ioctl_arg;
+    ioctl_arg.ic_cmd = ISPTM;
+    ioctl_arg.ic_timout = 0;
+    ioctl_arg.ic_len = 0;
+    ioctl_arg.ic_dp = NULL;
+
+    if (ioctl (fd, I_STR, &ioctl_arg) < 0)
+      {
+        errno = ENOTTY;
+        return errno;
+      }
+  }
+  {
+    char tmpbuf[9 + 10 + 1];
+    int n = sprintf (tmpbuf, "/dev/pts/%u", minor (st.st_rdev));
+    if (n >= buflen)
+      {
+        errno = ERANGE;
+        return errno;
+      }
+    memcpy (buf, tmpbuf, n + 1);
+  }
+#else
   if (!__isatty (fd))
     {
 #if ISATTY_FAILS_WITHOUT_SETTING_ERRNO && defined F_GETFL /* IRIX, Solaris */
@@ -85,6 +134,7 @@
     }
 
   buf[sizeof (_PATH_DEV) - 1] = 't';
+#endif
 
   if (__stat (buf, &st) < 0)
     return errno;




reply via email to

[Prev in Thread] Current Thread [Next in Thread]