aboutsummaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
Diffstat (limited to 'fs')
-rw-r--r--fs/overlayfs/dir.c24
1 files changed, 19 insertions, 5 deletions
diff --git a/fs/overlayfs/dir.c b/fs/overlayfs/dir.c
index d139405d2bfa..2578a0c0677d 100644
--- a/fs/overlayfs/dir.c
+++ b/fs/overlayfs/dir.c
@@ -506,11 +506,25 @@ static int ovl_remove_and_whiteout(struct dentry *dentry, bool is_dir)
506 struct dentry *opaquedir = NULL; 506 struct dentry *opaquedir = NULL;
507 int err; 507 int err;
508 508
509 if (is_dir && OVL_TYPE_MERGE_OR_LOWER(ovl_path_type(dentry))) { 509 if (is_dir) {
510 opaquedir = ovl_check_empty_and_clear(dentry); 510 if (OVL_TYPE_MERGE_OR_LOWER(ovl_path_type(dentry))) {
511 err = PTR_ERR(opaquedir); 511 opaquedir = ovl_check_empty_and_clear(dentry);
512 if (IS_ERR(opaquedir)) 512 err = PTR_ERR(opaquedir);
513 goto out; 513 if (IS_ERR(opaquedir))
514 goto out;
515 } else {
516 LIST_HEAD(list);
517
518 /*
519 * When removing an empty opaque directory, then it
520 * makes no sense to replace it with an exact replica of
521 * itself. But emptiness still needs to be checked.
522 */
523 err = ovl_check_empty_dir(dentry, &list);
524 ovl_cache_free(&list);
525 if (err)
526 goto out;
527 }
514 } 528 }
515 529
516 err = ovl_lock_rename_workdir(workdir, upperdir); 530 err = ovl_lock_rename_workdir(workdir, upperdir);