Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

kernel/vfs: Validate that only one of O_RDWR or O_WRONLY is used, if any.

Browse Source 
Otherwise we will have open modes that could have both and lead to
confusion in code that presumes only one will be set.

This catches the cause of some ported software (e.g. Wayland layer)
misbehaving with ramfs mounted in /var/shared_memory: the ramfs does not
properly handle both flags set, and due to another bug, they are for
shm_open'ed files.
This commit is contained in:
Augustin Cavalier 2022-12-07 17:56:27 -05:00
parent 2c6cf678e6
commit 9bc250e28d
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/system/kernel/fs/vfs.cpp
View File

@ -2831,6 +2831,9 @@ get_new_fd(int type, struct fs_mount* mount, struct vnode* vnode,
&& (type == FDTYPE_FILE || type == FDTYPE_DIR))
return B_BUSY;
if ((openMode & O_RDWR) != 0 && (openMode & O_WRONLY) != 0)
return B_BAD_VALUE;
descriptor = alloc_fd();
if (!descriptor)
return B_NO_MEMORY;