]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit
[JFFS2] Tidy up fix for ACL/permissions problem.
authorKaiGai Kohei <kaigai@ak.jp.nec.com>
Fri, 14 Sep 2007 06:16:35 +0000 (15:16 +0900)
committerDavid Woodhouse <dwmw2@infradead.org>
Sat, 20 Oct 2007 13:10:54 +0000 (14:10 +0100)
commitcfc8dc6f6f69ede939e09c2af06a01adee577285
treea37149d6353fb4b911beb5ad4ad18f2ec2b1468a
parente8b8c977734193adedf2b0f607d6252c78e86394
[JFFS2] Tidy up fix for ACL/permissions problem.

[In commit 9ed437c50d89eabae763dd422579f73fdebf288d we fixed a problem
with standard permissions on newly-created inodes, when POSIX ACLs are
enabled. This cleans it up...]

The attached patch separate jffs2_init_acl() into two parts.

The one is jffs2_init_acl_pre() called from jffs2_new_inode().
It compute ACL oriented inode->i_mode bits, and allocate in-memory ACL
objects associated with the new inode just before when inode meta
infomation is written to the medium.

The other is jffs2_init_acl_post() called from jffs2_symlink(),
jffs2_mkdir(), jffs2_mknod() and jffs2_do_create().
It actually writes in-memory ACL objects into the medium next to
the success of writing meta-information.

In the current implementation, we have to write a same inode meta
infomation twice when inode->i_mode is updated by the default ACL.
However, we can avoid the behavior by putting an updated i_mode
before it is written at first, as jffs2_init_acl_pre() doing.

Signed-off-by: KaiGai Kohei <kaigai@ak.jp.nec.com>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
fs/jffs2/acl.c
fs/jffs2/acl.h
fs/jffs2/dir.c
fs/jffs2/fs.c
fs/jffs2/os-linux.h
fs/jffs2/write.c