@@ -2514,11 +2514,11 @@ end_rename:
int ext4_permission(struct inode *inode, int mask)
{
-
#ifdef CONFIG_EXT4_FS_RICHACL
if (richacl_enabled(inode->i_sb))
return ext4_richacl_permission(inode,
- richacl_want_to_mask(mask));
+ richacl_want_to_mask(mask,
+ S_ISDIR(inode->i_mode)));
else
#endif
return generic_permission(inode, mask, ext4_check_acl);
@@ -188,16 +188,24 @@ EXPORT_SYMBOL_GPL(richacl_chmod);
* When checking for append, @want is (MAY_WRITE | MAY_APPEND).
*/
unsigned int
-richacl_want_to_mask(int want)
+richacl_want_to_mask(int want, int is_dir)
{
unsigned int mask = 0;
-
- if (want & MAY_READ)
- mask |= ACE4_READ_DATA;
- if (want & MAY_APPEND)
- mask |= ACE4_APPEND_DATA;
- else if (want & MAY_WRITE)
- mask |= ACE4_WRITE_DATA;
+ if (is_dir) {
+ if (want & MAY_READ)
+ mask |= ACE4_LIST_DIRECTORY;
+ if (want & MAY_APPEND)
+ mask |= ACE4_ADD_FILE | ACE4_ADD_SUBDIRECTORY;
+ else if (want & MAY_WRITE)
+ mask |= ACE4_ADD_FILE | ACE4_ADD_SUBDIRECTORY;
+ } else {
+ if (want & MAY_READ)
+ mask |= ACE4_READ_DATA;
+ if (want & MAY_APPEND)
+ mask |= ACE4_APPEND_DATA;
+ else if (want & MAY_WRITE)
+ mask |= ACE4_WRITE_DATA;
+ }
if (want & MAY_EXEC)
mask |= ACE4_EXECUTE;
@@ -222,7 +222,7 @@ richace_is_deny(const struct richace *ace)
extern struct richacl *richacl_alloc(int count);
extern struct richacl *richacl_clone(const struct richacl *acl);
-extern unsigned int richacl_want_to_mask(int want);
+extern unsigned int richacl_want_to_mask(int want, int is_dir);
extern int richacl_permission(struct inode *,
const struct richacl *, unsigned int);
extern int richacl_generic_permission(struct inode *, unsigned int);
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> --- fs/ext4/namei.c | 4 ++-- fs/richacl_base.c | 24 ++++++++++++++++-------- include/linux/richacl.h | 2 +- 3 files changed, 19 insertions(+), 11 deletions(-)