kernel: descrip: Only use copyinstr() in SYS_open

Signed-off-by: Ian Moffett <ian@osmora.org>

1 files changed, 13 insertions, 1 deletions

diff --git a/sys/kern/vfs_syscalls.c b/sys/kern/vfs_syscalls.c
index 0ef3d42..fc85759 100644
--- a/sys/kern/vfs_syscalls.c
+++ b/sys/kern/vfs_syscalls.c
@@ -83,6 +83,18 @@ vfs_dostat(const char *path, struct stat *sbuf)
     return 0;
 }
 
+static int
+vfs_doopen(const char *pathname, int flags)
+{
+    char pathbuf[PATH_MAX];
+
+    if (copyinstr(pathname, pathbuf, PATH_MAX) < 0) {
+        return -EFAULT;
+    }
+
+    return fd_open(pathbuf, flags);
+}
+
 /*
  * arg0: pathname
  * arg1: oflags
@@ -92,7 +104,7 @@ vfs_dostat(const char *path, struct stat *sbuf)
 scret_t
 sys_open(struct syscall_args *scargs)
 {
-    return fd_open((char *)scargs->arg0, scargs->arg1);
+    return vfs_doopen((char *)scargs->arg0, scargs->arg1);
 }
 
 /*