diff options
| author | dzwdz | 2022-10-02 19:25:17 +0200 |
|---|---|---|
| committer | dzwdz | 2022-10-02 19:25:17 +0200 |
| commit | 710e9b2b5c16f74f66420c66d12455ad518d42c7 (patch) | |
| tree | 82b88cc07ef3f122512354d649af68584bd4da4d /src/user/lib/fs/whitelist.c | |
| parent | 503d9ff758f8b83295830bdfc8c2ea56837d25e5 (diff) | |
syscall/open: add the full suite of READ/WRITE flags
Diffstat (limited to 'src/user/lib/fs/whitelist.c')
| -rw-r--r-- | src/user/lib/fs/whitelist.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/user/lib/fs/whitelist.c b/src/user/lib/fs/whitelist.c index 676b36f..571ebfb 100644 --- a/src/user/lib/fs/whitelist.c +++ b/src/user/lib/fs/whitelist.c @@ -1,5 +1,6 @@ #include <camellia/flags.h> #include <camellia/syscalls.h> +#include <errno.h> #include <stdlib.h> #include <string.h> #include <user/lib/fs/dir.h> @@ -49,6 +50,7 @@ void fs_whitelist(const char **whitelist) { switch (res.op) { case VFSOP_OPEN: { + bool error = false; bool passthru = false; bool inject = false; @@ -57,8 +59,9 @@ void fs_whitelist(const char **whitelist) { size_t entry_len = suffix_parse(*entry, strlen(*entry), &ro); /* If *entry is a prefix of the opened path, pass the open() through. */ if (prefix_match(*entry, entry_len, buf, res.len)) { - if (ro) res.flags |= OPEN_RO; passthru = true; + if (ro && OPEN_WRITEABLE(res.flags)) + error = true; break; } /* If the path is a prefix of *entry, we might need to inject a directory. */ @@ -66,7 +69,9 @@ void fs_whitelist(const char **whitelist) { inject = true; } } - if (passthru) { + if (error) { + _syscall_fs_respond(reqh, NULL, -EACCES, 0); + } else if (passthru) { forward_open(reqh, buf, res.len, res.flags); } else if (inject) { // TODO all the inject points could be precomputed |