diff options
| author | Daniel Rosenberg <drosen@google.com> | 2016-08-16 15:19:26 -0700 |
|---|---|---|
| committer | Daniel Rosenberg <drosen@google.com> | 2017-01-26 15:53:30 -0800 |
| commit | 96acdab2f4de226a253abbc6f7bab149bb82c4bf (patch) | |
| tree | d18f5ca20172c03ece37a06fe3fb420b4c524acf /fs/sdcardfs/inode.c | |
| parent | cc16efd13c3015889efe0bfe4f2cb5f4020ec4ca (diff) | |
ANDROID: sdcardfs: Fix locking for permission fix up
Iterating over d_subdirs requires taking d_lock.
Removed several unneeded locks.
Change-Id: I5b1588e54c7e6ee19b756d6705171c7f829e2650
Signed-off-by: Daniel Rosenberg <drosen@google.com>
Diffstat (limited to 'fs/sdcardfs/inode.c')
| -rw-r--r-- | fs/sdcardfs/inode.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/sdcardfs/inode.c b/fs/sdcardfs/inode.c index 4b140ba86955..1a23c0cc8f58 100644 --- a/fs/sdcardfs/inode.c +++ b/fs/sdcardfs/inode.c @@ -513,11 +513,9 @@ static int sdcardfs_rename(struct inode *old_dir, struct dentry *old_dentry, } /* At this point, not all dentry information has been moved, so * we pass along new_dentry for the name.*/ - mutex_lock(&d_inode(old_dentry)->i_mutex); get_derived_permission_new(new_dentry->d_parent, old_dentry, new_dentry); fix_derived_permission(d_inode(old_dentry)); get_derive_permissions_recursive(old_dentry); - mutex_unlock(&d_inode(old_dentry)->i_mutex); out: unlock_rename(lower_old_dir_dentry, lower_new_dir_dentry); dput(lower_old_dir_dentry); |