Don't allow acls which contain unmapped identifiers: they are meaningful
for remote file systems only.

Signed-off-by: Andreas Gruenbacher <agrue...@redhat.com>
---
 fs/ext4/richacl.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/fs/ext4/richacl.c b/fs/ext4/richacl.c
index 83b7a95..4ef140e 100644
--- a/fs/ext4/richacl.c
+++ b/fs/ext4/richacl.c
@@ -63,8 +63,13 @@ __ext4_set_richacl(handle_t *handle, struct inode *inode, 
struct richacl *acl)
        int retval;
 
        if (acl) {
-               mode_t mode = inode->i_mode;
+               mode_t mode;
 
+               /* Don't allow acls with unmapped identifiers. */
+               if (richacl_has_unmapped_identifiers(acl))
+                       return -EINVAL;
+
+               mode = inode->i_mode;
                if (richacl_equiv_mode(acl, &mode) == 0) {
                        inode->i_mode = mode;
                        ext4_mark_inode_dirty(handle, inode);
-- 
2.5.0

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to