diff options
| author |   Jan Kara <jack@suse.cz> | 2018-02-21 17:27:44 +0100 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2018-02-27 10:25:33 +0100 |
| commit | 70260e44750356fecb40ff5fcb0f91bcc911ab5f (patch) | |
| tree | 257292ff2eb539b1f69019d3269dd83743a93488 /fs/udf/inode.c | |
| parent | udf: Fix handling of Partition Descriptors (diff) | |
| download | linux-dev-70260e44750356fecb40ff5fcb0f91bcc911ab5f.tar.xz linux-dev-70260e44750356fecb40ff5fcb0f91bcc911ab5f.zip | |
udf: Ignore [ug]id=ignore mount options
Currently uid=ignore and gid=ignore make no sense without uid=<number>
and gid=<number> respectively as they result in all files having invalid
uid / gid which then doesn't allow even root to modify files and thus
causes confusion. And since commit ca76d2d8031f "UDF: fix UID and GID
mount option ignorance" (from over 10 years ago) uid=<number> overrides
all uids on disk as uid=ignore does. So just silently ignore uid=ignore
mount option.
Reviewed-by: Pali Rohár <pali.rohar@gmail.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/udf/inode.c')
| -rw-r--r-- | fs/udf/inode.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/udf/inode.c b/fs/udf/inode.c index c23744d5ae5c..9021c15cec17 100644 --- a/fs/udf/inode.c +++ b/fs/udf/inode.c @@ -1402,13 +1402,11 @@ reread: read_lock(&sbi->s_cred_lock); i_uid_write(inode, le32_to_cpu(fe->uid)); if (!uid_valid(inode->i_uid) || - UDF_QUERY_FLAG(inode->i_sb, UDF_FLAG_UID_IGNORE) || UDF_QUERY_FLAG(inode->i_sb, UDF_FLAG_UID_SET)) inode->i_uid = UDF_SB(inode->i_sb)->s_uid; i_gid_write(inode, le32_to_cpu(fe->gid)); if (!gid_valid(inode->i_gid) || - UDF_QUERY_FLAG(inode->i_sb, UDF_FLAG_GID_IGNORE) || UDF_QUERY_FLAG(inode->i_sb, UDF_FLAG_GID_SET)) inode->i_gid = UDF_SB(inode->i_sb)->s_gid; |