ANDROID: sdcardfs: remove unnecessary call to do_munmap
authorDaniel Rosenberg <drosen@google.com>
Fri, 10 Mar 2017 05:14:45 +0000 (21:14 -0800)
committerStricted <info@stricted.net>
Thu, 11 Oct 2018 16:03:35 +0000 (18:03 +0200)
Adapted from wrapfs
commit 5be6de9ecf02 ("Wrapfs: use vm_munmap in ->mmap")
commit 2c9f6014a8bb ("Wrapfs: remove unnecessary call
to vm_unmap in ->mmap")

Code is unnecessary and causes deadlocks in newer kernels.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
Signed-off-by: Daniel Rosenberg <drosen@google.com>
Bug: 35766959
Change-Id: Ia252d60c60799d7e28fc5f1f0f5b5ec2430a2379

fs/sdcardfs/file.c

index 42f66f2425bec88c65e596661646cf5e10464e51..93b93bec6a729e66d68183731a95ed3f7c72af5c 100644 (file)
@@ -176,12 +176,6 @@ static int sdcardfs_mmap(struct file *file, struct vm_area_struct *vma)
                        goto out;
                }
                saved_vm_ops = vma->vm_ops; /* save: came from lower ->mmap */
-               err = do_munmap(current->mm, vma->vm_start,
-                               vma->vm_end - vma->vm_start);
-               if (err) {
-                       pr_err("sdcardfs: do_munmap failed %d\n", err);
-                       goto out;
-               }
        }
 
        /*