sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Do not require write-access for fsync

Browse Source 
* fsync doesn't modify the file. It doesn't make sense to require
  write-access.

* This matches what ~all other systems do.
  At least Linux, macOS, and NetBSD.
This commit is contained in:
YAMAMOTO Takashi 2022-01-14 14:15:09 +09:00 committed by Petro Karashchenko
parent 3aed0aa641
commit 39ab0f4836
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
fs/vfs/fs_fsync.c
View File

@ -55,13 +55,6 @@ int file_fsync(FAR struct file *filep)
{ {
struct inode *inode; struct inode *inode;
/* Was this file opened for write access? */
if ((filep->f_oflags & O_WROK) == 0)
{
return -EBADF;
}
/* Is this inode a registered mountpoint? Does it support the /* Is this inode a registered mountpoint? Does it support the
* sync operations may be relevant to device drivers but only * sync operations may be relevant to device drivers but only
* the mountpoint operations vtable contains a sync method. * the mountpoint operations vtable contains a sync method.