Define SANITIZER_USES_CANONICAL_LINUX_SYSCALLS in compiler-rt

In defining this I noticed that musl will also choose to use
these canonical syscalls when the alternative is not defined.

This allows us to remove a bunch of duplicate syscalls and
their implementation.

Author: sbc100

src/library_syscall.js

@@ -300,17 +300,6 @@ var SyscallsLibrary = {
     return 0;
   },
 
-  __syscall_open: function(path, flags, varargs) {
-    var pathname = SYSCALLS.getStr(path);
-    var mode = varargs ? SYSCALLS.get() : 0;
-    var stream = FS.open(pathname, flags, mode);
-    return stream.fd;
-  },
-  __syscall_unlink: function(path) {
-    path = SYSCALLS.getStr(path);
-    FS.unlink(path);
-    return 0;
-  },
   __syscall_chdir: function(path) {
     path = SYSCALLS.getStr(path);
     FS.chdir(path);
@@ -329,12 +318,6 @@ var SyscallsLibrary = {
     path = SYSCALLS.getStr(path);
     return SYSCALLS.doAccess(path, amode);
   },
-  __syscall_rename: function(old_path, new_path) {
-    old_path = SYSCALLS.getStr(old_path);
-    new_path = SYSCALLS.getStr(new_path);
-    FS.rename(old_path, new_path);
-    return 0;
-  },
   __syscall_mkdir: function(path, mode) {
     path = SYSCALLS.getStr(path);
     return SYSCALLS.doMkdir(path, mode);
@@ -429,10 +412,6 @@ var SyscallsLibrary = {
     FS.symlink(target, linkpath);
     return 0;
   },
-  __syscall_readlink: function(path, buf, bufsize) {
-    path = SYSCALLS.getStr(path);
-    return SYSCALLS.doReadlink(path, buf, bufsize);
-  },
   __syscall_munmap__deps: ['$syscallMunmap'],
   __syscall_munmap: function(addr, len) {
     return syscallMunmap(addr, len);
@@ -982,9 +961,6 @@ var SyscallsLibrary = {
     return 0; // your advice is important to us (but we can't use it)
   },
   __syscall_openat: function(dirfd, path, flags, varargs) {
-#if SYSCALL_DEBUG
-    err('warning: untested syscall');
-#endif
     path = SYSCALLS.getStr(path);
     path = SYSCALLS.calculateAt(dirfd, path);
     var mode = varargs ? SYSCALLS.get() : 0;
@@ -1018,7 +994,7 @@ var SyscallsLibrary = {
     FS.chown(path, owner, group);
     return 0;
   },
-  __syscall_fstatat64: function(dirfd, path, buf, flags) {
+  __syscall_newfstatat: function(dirfd, path, buf, flags) {
     path = SYSCALLS.getStr(path);
     var nofollow = flags & {{{ cDefine('AT_SYMLINK_NOFOLLOW') }}};
     var allowEmpty = flags & {{{ cDefine('AT_EMPTY_PATH') }}};
@@ -1042,9 +1018,6 @@ var SyscallsLibrary = {
     return 0;
   },
   __syscall_renameat: function(olddirfd, oldpath, newdirfd, newpath) {
-#if SYSCALL_DEBUG
-    err('warning: untested syscall');
-#endif
     oldpath = SYSCALLS.getStr(oldpath);
     newpath = SYSCALLS.getStr(newpath);
     oldpath = SYSCALLS.calculateAt(olddirfd, oldpath);
@@ -1066,9 +1039,6 @@ var SyscallsLibrary = {
     return 0;
   },
   __syscall_readlinkat: function(dirfd, path, buf, bufsize) {
-#if SYSCALL_DEBUG
-    err('warning: untested syscall');
-#endif
     path = SYSCALLS.getStr(path);
     path = SYSCALLS.calculateAt(dirfd, path);
     return SYSCALLS.doReadlink(path, buf, bufsize);

system/lib/compiler-rt/lib/sanitizer_common/sanitizer_platform.h

@@ -286,6 +286,11 @@
 #define SANITIZER_SIGN_EXTENDED_ADDRESSES 0
 #endif
 
+// Emscripten emulates the canonical linux syscall set.
+#if !defined SANITIZER_USES_CANONICAL_LINUX_SYSCALLS && SANITIZER_EMSCRIPTEN
+# define SANITIZER_USES_CANONICAL_LINUX_SYSCALLS 1
+#endif
+
 // The AArch64 and RISC-V linux ports use the canonical syscall set as
 // mandated by the upstream linux community for all new ports. Other ports
 // may still use legacy syscalls.

system/lib/fetch/asmfs.cpp

@@ -1206,7 +1206,7 @@ long __syscall_link(long oldpath, long newpath)
   RETURN_ERRNO(ENOTSUP, "TODO: link() is a stub and not yet implemented in ASMFS");
 }
 
-long __syscall_unlink(long path)
+long __syscall_unlinkat(long path, long flags)
 {
   const char* pathname = (const char *)path;
 #ifdef ASMFS_DEBUG

system/lib/libc/musl/arch/emscripten/bits/syscall.h

@@ -1,6 +1,4 @@
-#define SYS_open		  __syscall_open
 #define SYS_link		  __syscall_link
-#define SYS_unlink		 __syscall_unlink
 #define SYS_chdir		 __syscall_chdir
 #define SYS_mknod		 __syscall_mknod
 #define SYS_chmod		 __syscall_chmod
@@ -9,7 +7,6 @@
 #define SYS_access		 __syscall_access
 #define SYS_nice		 __syscall_nice
 #define SYS_sync		 __syscall_sync
-#define SYS_rename		 __syscall_rename
 #define SYS_mkdir		 __syscall_mkdir
 #define SYS_rmdir		 __syscall_rmdir
 #define SYS_dup		 __syscall_dup
@@ -24,7 +21,6 @@
 #define SYS_setrlimit		 __syscall_setrlimit
 #define SYS_getrusage		 __syscall_getrusage
 #define SYS_symlink		 __syscall_symlink
-#define SYS_readlink		 __syscall_readlink
 #define SYS_munmap		 __syscall_munmap
 #define SYS_fchmod		 __syscall_fchmod
 #define SYS_getpriority	 __syscall_getpriority
@@ -76,7 +72,7 @@
 #define SYS_mkdirat		__syscall_mkdirat
 #define SYS_mknodat		__syscall_mknodat
 #define SYS_fchownat		__syscall_fchownat
-#define SYS_fstatat64		__syscall_fstatat64
+#define SYS_newfstatat		__syscall_newfstatat
 #define SYS_unlinkat		__syscall_unlinkat
 #define SYS_renameat		__syscall_renameat
 #define SYS_linkat		__syscall_linkat

system/lib/libc/musl/arch/emscripten/syscall_arch.h

@@ -11,9 +11,7 @@
 extern "C" {
 #endif
 
-long __syscall_open(long path, long flags, ...); // mode is optional
 long __syscall_link(long oldpath, long newpath);
-long __syscall_unlink(long path);
 long __syscall_chdir(long path);
 long __syscall_mknod(long path, long mode, long dev);
 long __syscall_chmod(long path, long mode);
@@ -22,7 +20,6 @@ long __syscall_pause(void);
 long __syscall_access(long path, long amode);
 long __syscall_nice(long inc);
 long __syscall_sync(void);
-long __syscall_rename(long old_path, long new_path);
 long __syscall_mkdir(long path, long mode);
 long __syscall_rmdir(long path);
 long __syscall_dup(long fd);
@@ -38,7 +35,6 @@ long __syscall_setsid(void);
 long __syscall_setrlimit(long resource, long limit);
 long __syscall_getrusage(long who, long usage);
 long __syscall_symlink(long target, long linkpath);
-long __syscall_readlink(long path, long buf, long bufsize);
 long __syscall_munmap(long addr, long len);
 long __syscall_fchmod(long fd, long mode);
 long __syscall_getpriority(long which, long who);
@@ -93,11 +89,11 @@ long __syscall_fcntl64(long fd, long cmd, ...);
 long __syscall_statfs64(long path, long size, long buf);
 long __syscall_fstatfs64(long fd, long size, long buf);
 long __syscall_fadvise64_64(long fd, long low, long high, long low2, long high2, long advice);
-long __syscall_openat(long dirfd, long path, long flags, ...);
+long __syscall_openat(long dirfd, long path, long flags, ...); // mode is optional
 long __syscall_mkdirat(long dirfd, long path, long mode);
 long __syscall_mknodat(long dirfd, long path, long mode, long dev);
 long __syscall_fchownat(long dirfd, long path, long owner, long group, long flags);
-long __syscall_fstatat64(long dirfd, long path, long buf, long flags);
+long __syscall_newfstatat(long dirfd, long path, long buf, long flags);
 long __syscall_unlinkat(long dirfd, long path, long flags);
 long __syscall_renameat(long olddirfd, long oldpath, long newdirfd, long newpath);
 long __syscall_linkat(long olddirfd, long oldpath, long newdirfd, long newpath, long flags);

system/lib/libc/musl/src/internal/syscall.h

@@ -257,10 +257,10 @@ long __syscall_ret(unsigned long),
 #define __sys_open_cp3(x,pn,fl,mo) __syscall_cp4(SYS_openat, AT_FDCWD, pn, (fl)|O_LARGEFILE, mo)
 #endif
 #else // __EMSCRIPTEN__
-#define __sys_open2(x,pn,fl) __syscall_open(__scc(pn), __scc((fl)|O_LARGEFILE))
-#define __sys_open3(x,pn,fl,mo) __syscall_open(__scc(pn), __scc((fl)|O_LARGEFILE), __scc(mo))
-#define __sys_open_cp2(x,pn,fl) __syscall_open(__scc(pn), __scc((fl)|O_LARGEFILE))
-#define __sys_open_cp3(x,pn,fl,mo) __syscall_open(__scc(pn), __scc((fl)|O_LARGEFILE), __scc(mo))
+#define __sys_open2(x,pn,fl) __syscall_openat(__scc(AT_FDCWD), __scc(pn), __scc((fl)|O_LARGEFILE))
+#define __sys_open3(x,pn,fl,mo) __syscall_openat(__scc(AT_FDCWD), __scc(pn), __scc((fl)|O_LARGEFILE), __scc(mo))
+#define __sys_open_cp2(x,pn,fl) __syscall_openat(__scc(AT_FDCWD), __scc(pn), __scc((fl)|O_LARGEFILE))
+#define __sys_open_cp3(x,pn,fl,mo) __syscall_openat(__scc(AT_FDCWD), __scc(pn), __scc((fl)|O_LARGEFILE), __scc(mo))
 #endif
 #define __sys_open(...) __SYSCALL_DISP(__sys_open,,__VA_ARGS__)
 #define sys_open(...) __syscall_ret(__sys_open(__VA_ARGS__))

system/lib/standalone/standalone.c

@@ -70,7 +70,7 @@ long __syscall_mmap2(long addr, long len, long prot, long flags, long fd, long o
 // corner case error checking; everything else is not permitted.
 // TODO: full file support for WASI, or an option for it
 // open()
-long __syscall_open(const char* path, long flags, ...) {
+long __syscall_openat(const char* path, long flags, ...) {
   if (!strcmp(path, "/dev/stdin")) return STDIN_FILENO;
   if (!strcmp(path, "/dev/stdout")) return STDOUT_FILENO;
   if (!strcmp(path, "/dev/stderr")) return STDERR_FILENO;

system/lib/wasmfs/syscalls.cpp

@@ -282,7 +282,7 @@ long __syscall_fstat64(long fd, long buf) {
   return __WASI_ERRNO_SUCCESS;
 }
 
-__wasi_fd_t __syscall_open(long pathname, long flags, long mode) {
+__wasi_fd_t __syscall_openat(int dirfd, long pathname, long flags, long mode) {
   int accessMode = (flags & O_ACCMODE);
   bool canWrite = false;