Native implementation of ctermid()

Author: otethal

graalpython/com.oracle.graal.python.cext/posix/posix.c

@@ -479,6 +479,10 @@ int64_t call_getsid(int64_t pid) {
     return getsid(pid);
 }
 
+int32_t call_ctermid(char *buf) {
+    return ctermid(buf) == NULL ? -1 : 0;
+}
+
 int32_t get_errno() {
     return errno;
 }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/PosixModuleBuiltins.java

@@ -439,7 +439,7 @@ protected ArgumentClinicProvider getArgumentClinic() {
 
         @Specialization
         long getSid(VirtualFrame frame, long pid,
-                    @CachedLibrary("getPosixSupport()") PosixSupportLibrary posixLib) {
+                        @CachedLibrary("getPosixSupport()") PosixSupportLibrary posixLib) {
             try {
                 return posixLib.getsid(getPosixSupport(), pid);
             } catch (PosixException e) {
@@ -1989,8 +1989,13 @@ int umask(VirtualFrame frame, int mask,
     @GenerateNodeFactory
     abstract static class CtermId extends PythonBuiltinNode {
         @Specialization
-        static String ctermid() {
-            return "/dev/tty";
+        String ctermid(VirtualFrame frame,
+                        @CachedLibrary("getPosixSupport()") PosixSupportLibrary posixLib) {
+            try {
+                return posixLib.ctermid(getPosixSupport());
+            } catch (PosixException e) {
+                throw raiseOSErrorFromPosixException(frame, e);
+            }
         }
     }
 

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/EmulatedPosixSupport.java

@@ -1584,6 +1584,12 @@ public long getsid(long pid) {
         throw new UnsupportedPosixFeatureException("Emulated getsid not supported");
     }
 
+    @ExportMessage
+    @SuppressWarnings("static-method")
+    public String ctermid() {
+        return "/dev/tty";
+    }
+
     @ExportMessage
     @TruffleBoundary
     public int forkExec(Object[] executables, Object[] args, Object cwd, Object[] env, int stdinReadFd, int stdinWriteFd, int stdoutReadFd, int stdoutWriteFd, int stderrReadFd, int stderrWriteFd,

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/LoggingPosixSupport.java

@@ -686,6 +686,16 @@ final long getsid(long pid,
         }
     }
 
+    @ExportMessage
+    final String ctermid(@CachedLibrary("this.delegate") PosixSupportLibrary lib) throws PosixException {
+        logEnter("ctermid", "");
+        try {
+            return logExit("ctermid", "%s", lib.ctermid(delegate));
+        } catch (PosixException e) {
+            throw logException("ctermid", e);
+        }
+    }
+
     @ExportMessage
     final int forkExec(Object[] executables, Object[] args, Object cwd, Object[] env, int stdinReadFd, int stdinWriteFd, int stdoutReadFd, int stdoutWriteFd, int stderrReadFd, int stderrWriteFd,
                     int errPipeReadFd, int errPipeWriteFd, boolean closeFds, boolean restoreSignals, boolean callSetsid, int[] fdsToKeep,

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/NFIPosixSupport.java

@@ -89,6 +89,7 @@ public final class NFIPosixSupport extends PosixSupport {
     private static final int UNAME_BUF_LENGTH = 256;
     private static final int DIRENT_NAME_BUF_LENGTH = 256;
     private static final int PATH_MAX = 4096;
+    private static final int L_ctermid = 9;
 
     private static final int MAX_READ = Integer.MAX_VALUE / 2;
 
@@ -151,6 +152,7 @@ private enum PosixNativeFunction {
         call_getuid("():sint64"),
         call_getppid("():sint64"),
         call_getsid("(sint64):sint64"),
+        call_ctermid("([sint8]):sint32"),
         fork_exec("([sint8], [sint64], sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, sint32, [sint32], sint64):sint32");
 
         private final String signature;
@@ -913,6 +915,17 @@ public long getsid(long pid,
         return res;
     }
 
+    @ExportMessage
+    public String ctermid(@Shared("invoke") @Cached InvokeNativeFunction invokeNode) throws PosixException {
+        byte[] buf = new byte[L_ctermid];
+        int res = invokeNode.callInt(this, PosixNativeFunction.call_ctermid, wrap(buf));
+        if (res == -1) {
+            throw getErrnoAndThrowPosixException(invokeNode);
+        }
+        // TODO PyUnicode_DecodeFSDefault
+        return cStringToJavaString(buf);
+    }
+
     @ExportMessage
     public int forkExec(Object[] executables, Object[] args, Object cwd, Object[] env, int stdinReadFd, int stdinWriteFd, int stdoutReadFd, int stdoutWriteFd, int stderrReadFd, int stderrWriteFd,
                     int errPipeReadFd, int errPipeWriteFd, boolean closeFds, boolean restoreSignals, boolean callSetsid, int[] fdsToKeep,

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/PosixSupportLibrary.java

@@ -271,6 +271,8 @@ public abstract class PosixSupportLibrary extends Library {
 
     public abstract long getsid(Object receiver, long pid) throws PosixException;
 
+    public abstract String ctermid(Object receiver) throws PosixException;
+
     public abstract int forkExec(Object receiver, Object[] executables, Object[] args, Object cwd, Object[] env, int stdinReadFd, int stdinWriteFd, int stdoutReadFd, int stdoutWriteFd,
                     int stderrReadFd, int stderrWriteFd, int errPipeReadFd, int errPipeWriteFd, boolean closeFds, boolean restoreSignals, boolean callSetsid, int[] fdsToKeep) throws PosixException;