WasmFS JS API: prefer const char*

Author: kleisauke

system/lib/wasmfs/js_api.cpp

@@ -31,7 +31,7 @@ int wasmfs_create_directory(const char* path, int mode, backend_t backend);
 // Return a pointer to the JS buffer in HEAPU8.
 // The buffer will also contain the file length.
 // TODO: Use WasmFS ErrnoError handling instead of aborting on failure.
-void* _wasmfs_read_file(char* path) {
+void* _wasmfs_read_file(const char* path) {
   static_assert(sizeof(off_t) == 8, "File offset type must be 64-bit");
 
   struct stat file;
@@ -73,7 +73,7 @@ void* _wasmfs_read_file(char* path) {
 
 // Writes to a file, possibly creating it, and returns the number of bytes
 // written successfully. If the file already exists, appends to it.
-int _wasmfs_write_file(char* pathname, char* data, size_t data_size) {
+int _wasmfs_write_file(const char* pathname, const char* data, size_t data_size) {
   auto parsedParent = path::parseParent(pathname);
   if (parsedParent.getError()) {
     return 0;
@@ -123,37 +123,37 @@ int _wasmfs_write_file(char* pathname, char* data, size_t data_size) {
   return data_size;
 }
 
-int _wasmfs_mkdir(char* path, mode_t mode) {
+int _wasmfs_mkdir(const char* path, mode_t mode) {
   return __syscall_mkdirat(AT_FDCWD, path, mode);
 }
 
-int _wasmfs_rmdir(char* path) {
+int _wasmfs_rmdir(const char* path) {
   return __syscall_unlinkat(AT_FDCWD, path, AT_REMOVEDIR);
 }
 
-int _wasmfs_open(char* path, int flags, mode_t mode) {
+int _wasmfs_open(const char* path, int flags, mode_t mode) {
   return __syscall_openat(AT_FDCWD, path, flags, mode);
 }
 
 int _wasmfs_allocate(int fd, off_t offset, off_t len) {
   return __syscall_fallocate(fd, 0, offset, len);
 }
 
-int _wasmfs_mknod(char* path, mode_t mode, dev_t dev) {
+int _wasmfs_mknod(const char* path, mode_t mode, dev_t dev) {
   return __syscall_mknodat(AT_FDCWD, path, mode, dev);
 }
 
-int _wasmfs_unlink(char* path) {
+int _wasmfs_unlink(const char* path) {
   return __syscall_unlinkat(AT_FDCWD, path, 0);
 }
 
-int _wasmfs_chdir(char* path) { return __syscall_chdir(path); }
+int _wasmfs_chdir(const char* path) { return __syscall_chdir(path); }
 
-int _wasmfs_symlink(char* old_path, char* new_path) {
+int _wasmfs_symlink(const char* old_path, const char* new_path) {
   return __syscall_symlinkat(old_path, AT_FDCWD, new_path);
 }
 
-intptr_t _wasmfs_readlink(char* path) {
+intptr_t _wasmfs_readlink(const char* path) {
   static thread_local void* readBuf = nullptr;
   readBuf = realloc(readBuf, PATH_MAX);
   int bytes =
@@ -191,13 +191,13 @@ int _wasmfs_pwrite(int fd, void* buf, size_t count, off_t offset) {
   return numBytes;
 }
 
-int _wasmfs_chmod(char* path, mode_t mode) {
+int _wasmfs_chmod(const char* path, mode_t mode) {
   return __syscall_chmod(path, mode);
 }
 
 int _wasmfs_fchmod(int fd, mode_t mode) { return __syscall_fchmod(fd, mode); }
 
-int _wasmfs_lchmod(char* path, mode_t mode) {
+int _wasmfs_lchmod(const char* path, mode_t mode) {
   return __syscall_fchmodat2(AT_FDCWD, path, mode, AT_SYMLINK_NOFOLLOW);
 }
 
@@ -210,7 +210,7 @@ int _wasmfs_llseek(int fd, off_t offset, int whence) {
   return newOffset;
 }
 
-int _wasmfs_rename(char* oldpath, char* newpath) {
+int _wasmfs_rename(const char* oldpath, const char* newpath) {
   return __syscall_renameat(AT_FDCWD, oldpath, AT_FDCWD, newpath);
 }
 
@@ -240,7 +240,7 @@ int _wasmfs_pread(int fd, void* buf, size_t count, off_t offset) {
   return numBytes;
 }
 
-int _wasmfs_truncate(char* path, off_t length) {
+int _wasmfs_truncate(const char* path, off_t length) {
   return __syscall_truncate64(path, length);
 }
 
@@ -262,7 +262,7 @@ int _wasmfs_munmap(void* addr, size_t length) {
   return __syscall_munmap(addr, length);
 }
 
-int _wasmfs_utime(char* path, long atime_ms, long mtime_ms) {
+int _wasmfs_utime(const char* path, long atime_ms, long mtime_ms) {
   struct timespec times[2];
   times[0].tv_sec = atime_ms / 1000;
   times[0].tv_nsec = (atime_ms % 1000) * 1000000;
@@ -272,18 +272,18 @@ int _wasmfs_utime(char* path, long atime_ms, long mtime_ms) {
   return __syscall_utimensat(AT_FDCWD, path, times, 0);
 }
 
-int _wasmfs_stat(char* path, struct stat* statBuf) {
+int _wasmfs_stat(const char* path, struct stat* statBuf) {
   return __syscall_stat64(path, statBuf);
 }
 
-int _wasmfs_lstat(char* path, struct stat* statBuf) {
+int _wasmfs_lstat(const char* path, struct stat* statBuf) {
   return __syscall_lstat64(path, statBuf);
 }
 
 // The legacy JS API requires a mountpoint to already exist, so  WasmFS will
 // attempt to remove the target directory if it exists before replacing it with
 // a mounted directory.
-int _wasmfs_mount(char* path, wasmfs::backend_t created_backend) {
+int _wasmfs_mount(const char* path, wasmfs::backend_t created_backend) {
   int err = __syscall_rmdir(path);
 
   // The legacy JS API mount requires the directory to already exist, but we
@@ -299,7 +299,7 @@ int _wasmfs_mount(char* path, wasmfs::backend_t created_backend) {
 //   ENOENT - if nothing exists there
 //   EISDIR - if it is a directory
 //   EEXIST - if it is a normal file
-int _wasmfs_identify(char* path) {
+int _wasmfs_identify(const char* path) {
   struct stat file;
   int err = 0;
   err = stat(path, &file);
@@ -318,7 +318,7 @@ struct wasmfs_readdir_state {
   struct dirent** entries;
 };
 
-struct wasmfs_readdir_state* _wasmfs_readdir_start(char* path) {
+struct wasmfs_readdir_state* _wasmfs_readdir_start(const char* path) {
   struct dirent** entries;
   int nentries = scandir(path, &entries, NULL, alphasort);
   if (nentries == -1) {