bsd unlockpt: unlockpt needs to fail with EINVAL, not ENOTTY
authorSamuel Thibault <samuel.thibault@ens-lyon.org>
Sun, 8 Nov 2020 14:06:56 +0000 (14:06 +0000)
committerSamuel Thibault <samuel.thibault@ens-lyon.org>
Sun, 8 Nov 2020 14:06:56 +0000 (14:06 +0000)
The EINVAL error code is mandated by POSIX, while ptsname_r returns
ENOTTY, so we need to translate.

sysdeps/unix/bsd/unlockpt.c

index d8b7f0c..e609685 100644 (file)
@@ -20,6 +20,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
+#include <errno.h>
 
 
 /* Unlock the slave pseudo terminal associated with the master pseudo
@@ -31,6 +32,10 @@ unlockpt (int fd)
 
   /* BSD doesn't have a lock, but it does have `revoke'.  */
   if (__ptsname_r (fd, buf, sizeof (buf)))
-    return -1;
+    {
+      if (errno == ENOTTY)
+       __set_errno (EINVAL);
+      return -1;
+    }
   return __revoke (buf);
 }