* [PATCH] ubifs: atomically set inode->i_flags in ubifs_set_inode_flags() [not found] <1397465159-45836-1-git-send-email-zhenzhang.zhang@huawei.com> @ 2014-04-15 6:15 ` Zhang Zhen 2014-04-15 13:31 ` Artem Bityutskiy 0 siblings, 1 reply; 3+ messages in thread From: Zhang Zhen @ 2014-04-15 6:15 UTC (permalink / raw) To: dedekind1, adrian.hunter; +Cc: linux-mtd Use set_mask_bits() to atomically set i_flags instead of clearing out the S_IMMUTABLE, S_APPEND, etc. flags and then setting them from the UBIFS_IMMUTABLE, UBIFS_APPEND_FL, etc. flags, since this opens up a race where an immutable file has the immutable flag cleared for a brief window of time. Signed-off-by: Zhang Zhen <zhenzhang.zhang@huawei.com> --- fs/ubifs/ioctl.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/fs/ubifs/ioctl.c b/fs/ubifs/ioctl.c index 648b143..5d6df97 100644 --- a/fs/ubifs/ioctl.c +++ b/fs/ubifs/ioctl.c @@ -27,6 +27,7 @@ #include <linux/compat.h> #include <linux/mount.h> #include "ubifs.h" +#include <linux/bitops.h> /** * ubifs_set_inode_flags - set VFS inode flags. @@ -37,16 +38,18 @@ void ubifs_set_inode_flags(struct inode *inode) { unsigned int flags = ubifs_inode(inode)->flags; + unsigned int new_fl = 0; - inode->i_flags &= ~(S_SYNC | S_APPEND | S_IMMUTABLE | S_DIRSYNC); if (flags & UBIFS_SYNC_FL) - inode->i_flags |= S_SYNC; + new_fl |= S_SYNC; if (flags & UBIFS_APPEND_FL) - inode->i_flags |= S_APPEND; + new_fl |= S_APPEND; if (flags & UBIFS_IMMUTABLE_FL) - inode->i_flags |= S_IMMUTABLE; + new_fl |= S_IMMUTABLE; if (flags & UBIFS_DIRSYNC_FL) - inode->i_flags |= S_DIRSYNC; + new_fl |= S_DIRSYNC; + set_mask_bits(&inode->i_flags, + S_SYNC|S_APPEND|S_IMMUTABLE|S_DIRSYNC, new_fl); } /* -- 1.8.5.5 . ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ubifs: atomically set inode->i_flags in ubifs_set_inode_flags() 2014-04-15 6:15 ` [PATCH] ubifs: atomically set inode->i_flags in ubifs_set_inode_flags() Zhang Zhen @ 2014-04-15 13:31 ` Artem Bityutskiy 2014-04-16 2:09 ` Zhang Zhen 0 siblings, 1 reply; 3+ messages in thread From: Artem Bityutskiy @ 2014-04-15 13:31 UTC (permalink / raw) To: Zhang Zhen; +Cc: linux-mtd, adrian.hunter On Tue, 2014-04-15 at 14:15 +0800, Zhang Zhen wrote: > Use set_mask_bits() to atomically set i_flags instead of clearing out the > S_IMMUTABLE, S_APPEND, etc. flags and then setting them from the > UBIFS_IMMUTABLE, UBIFS_APPEND_FL, etc. flags, since this opens up a race > where an immutable file has the immutable flag cleared for a brief > window of time. > > Signed-off-by: Zhang Zhen <zhenzhang.zhang@huawei.com> Looks correct to me. Does it fix a real-life bug or just a hypothetical issue? Also, did you test it? -- Best Regards, Artem Bityutskiy ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ubifs: atomically set inode->i_flags in ubifs_set_inode_flags() 2014-04-15 13:31 ` Artem Bityutskiy @ 2014-04-16 2:09 ` Zhang Zhen 0 siblings, 0 replies; 3+ messages in thread From: Zhang Zhen @ 2014-04-16 2:09 UTC (permalink / raw) To: dedekind1; +Cc: linux-mtd, adrian.hunter On 2014/4/15 21:31, Artem Bityutskiy wrote: > On Tue, 2014-04-15 at 14:15 +0800, Zhang Zhen wrote: >> Use set_mask_bits() to atomically set i_flags instead of clearing out the >> S_IMMUTABLE, S_APPEND, etc. flags and then setting them from the >> UBIFS_IMMUTABLE, UBIFS_APPEND_FL, etc. flags, since this opens up a race >> where an immutable file has the immutable flag cleared for a brief >> window of time. >> >> Signed-off-by: Zhang Zhen <zhenzhang.zhang@huawei.com> > > Looks correct to me. Does it fix a real-life bug or just a hypothetical > issue? Also, did you test it? > Hi, It's just a hypothetical issue. I noticed the commit 00a1a053ebe5febcfc2ec498bd894f035ad2aa06 added the function set_mask_bits() to atomically set i_flags, and it is also useful to UBIFS. So i fix it. Sorry, Because of lacking of test board, i can only guarantee that it has no compile error. Best regards! ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-04-16 2:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1397465159-45836-1-git-send-email-zhenzhang.zhang@huawei.com>
2014-04-15 6:15 ` [PATCH] ubifs: atomically set inode->i_flags in ubifs_set_inode_flags() Zhang Zhen
2014-04-15 13:31 ` Artem Bityutskiy
2014-04-16 2:09 ` Zhang Zhen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).