diff options
Diffstat (limited to 'src/kernel/syscalls.c')
| -rw-r--r-- | src/kernel/syscalls.c | 86 |
1 files changed, 47 insertions, 39 deletions
diff --git a/src/kernel/syscalls.c b/src/kernel/syscalls.c index cfd895e..c916e81 100644 --- a/src/kernel/syscalls.c +++ b/src/kernel/syscalls.c @@ -148,8 +148,9 @@ int _syscall_mount(handle_t hid, const char __user *path, int len) { } if (hid >= 0) { // mounting a real backend? - struct handle *handle = process_handle_get(process_current, hid, HANDLE_FS_FRONT); - if (!handle) goto fail; + struct handle *handle = process_handle_get(process_current, hid); + if (!handle || handle->type != HANDLE_FS_FRONT) + goto fail; backend = handle->backend; backend->refcount++; } // otherwise backend == NULL @@ -176,48 +177,55 @@ fail: } int _syscall_read(handle_t handle_num, void __user *buf, size_t len, int offset) { - struct handle *h; - // TODO get rid of the type argument in process_handle_get - if ((h = process_handle_get(process_current, handle_num, HANDLE_FILE))) { - vfsreq_create((struct vfs_request) { - .type = VFSOP_READ, - .output = { - .buf = (userptr_t) buf, - .len = len, - }, - .id = h->file_id, - .offset = offset, - .caller = process_current, - .backend = h->backend, - }); - } else if ((h = process_handle_get(process_current, handle_num, HANDLE_PIPE))) { - pipe_joinqueue(h, true, process_current, buf, len); - pipe_trytransfer(h); - } else { - SYSCALL_RETURN(-1); + struct handle *h = process_handle_get(process_current, handle_num); + if (!h) SYSCALL_RETURN(-1); + switch (h->type) { + case HANDLE_FILE: + vfsreq_create((struct vfs_request) { + .type = VFSOP_READ, + .output = { + .buf = (userptr_t) buf, + .len = len, + }, + .id = h->file_id, + .offset = offset, + .caller = process_current, + .backend = h->backend, + }); + break; + case HANDLE_PIPE: + pipe_joinqueue(h, true, process_current, buf, len); + pipe_trytransfer(h); + break; + default: + SYSCALL_RETURN(-1); } return -1; // dummy } int _syscall_write(handle_t handle_num, const void __user *buf, size_t len, int offset) { - struct handle *h; - if ((h = process_handle_get(process_current, handle_num, HANDLE_FILE))) { - vfsreq_create((struct vfs_request) { - .type = VFSOP_WRITE, - .input = { - .buf = (userptr_t) buf, - .len = len, - }, - .id = h->file_id, - .offset = offset, - .caller = process_current, - .backend = h->backend, - }); - } else if ((h = process_handle_get(process_current, handle_num, HANDLE_PIPE))) { - pipe_joinqueue(h, false, process_current, (void __user *)buf, len); - pipe_trytransfer(h); - } else { - SYSCALL_RETURN(-1); + struct handle *h = process_handle_get(process_current, handle_num); + if (!h) SYSCALL_RETURN(-1); + switch (h->type) { + case HANDLE_FILE: + vfsreq_create((struct vfs_request) { + .type = VFSOP_WRITE, + .input = { + .buf = (userptr_t) buf, + .len = len, + }, + .id = h->file_id, + .offset = offset, + .caller = process_current, + .backend = h->backend, + }); + break; + case HANDLE_PIPE: + pipe_joinqueue(h, false, process_current, (void __user *)buf, len); + pipe_trytransfer(h); + break; + default: + SYSCALL_RETURN(-1); } return -1; // dummy } |