diff options
author | OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> | 2008-06-13 00:21:28 +0200 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-06-13 03:05:39 +0200 |
commit | 2d518f84e5ecd1d71df0e6ac5176d212f68c27ce (patch) | |
tree | fd5c4e3b350f82e516c63349e17e43805c0899fb /fs/char_dev.c | |
parent | mm: fix incorrect variable type in do_try_to_free_pages() (diff) | |
download | linux-2d518f84e5ecd1d71df0e6ac5176d212f68c27ce.tar.xz linux-2d518f84e5ecd1d71df0e6ac5176d212f68c27ce.zip |
fat: relax the permission check of fat_setattr()
New chmod() allows only acceptable permission, and if not acceptable, it
returns -EPERM. Old one allows even if it can't store permission to on
disk inode. But it seems too strict for users.
E.g. https://bugzilla.redhat.com/show_bug.cgi?id=449080: With new one,
rsync couldn't create the temporary file.
So, this patch allows like old one, but now it doesn't change the
permission if it can't store, and it returns 0.
Also, this patch fixes missing check.
Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/char_dev.c')
0 files changed, 0 insertions, 0 deletions