lkml.org 
[lkml]   [2016]   [Jan]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.19.y-ckt 020/210] ovl: check dentry positiveness in ovl_cleanup_whiteouts()
    Date
    3.19.8-ckt14 -stable review patch.  If anyone has any objections, please let me know.

    ---8<------------------------------------------------------------

    From: Konstantin Khlebnikov <khlebnikov@yandex-team.ru>

    commit 84889d49335627bc770b32787c1ef9ebad1da232 upstream.

    This patch fixes kernel crash at removing directory which contains
    whiteouts from lower layers.

    Cache of directory content passed as "list" contains entries from all
    layers, including whiteouts from lower layers. So, lookup in upper dir
    (moved into work at this stage) will return negative entry. Plus this
    cache is filled long before and we can race with external removal.

    Example:
    mkdir -p lower0/dir lower1/dir upper work overlay
    touch lower0/dir/a lower0/dir/b
    mknod lower1/dir/a c 0 0
    mount -t overlay none overlay -o lowerdir=lower1:lower0,upperdir=upper,workdir=work
    rm -fr overlay/dir

    Signed-off-by: Konstantin Khlebnikov <khlebnikov@yandex-team.ru>
    Signed-off-by: Miklos Szeredi <miklos@szeredi.hu>
    Signed-off-by: Kamal Mostafa <kamal@canonical.com>
    ---
    fs/overlayfs/readdir.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    diff --git a/fs/overlayfs/readdir.c b/fs/overlayfs/readdir.c
    index c020599..7733883 100644
    --- a/fs/overlayfs/readdir.c
    +++ b/fs/overlayfs/readdir.c
    @@ -581,7 +581,8 @@ void ovl_cleanup_whiteouts(struct dentry *upper, struct list_head *list)
    (int) PTR_ERR(dentry));
    continue;
    }
    - ovl_cleanup(upper->d_inode, dentry);
    + if (dentry->d_inode)
    + ovl_cleanup(upper->d_inode, dentry);
    dput(dentry);
    }
    mutex_unlock(&upper->d_inode->i_mutex);
    --
    1.9.1
    \
     
     \ /
      Last update: 2016-01-29 03:41    [W:4.026 / U:0.260 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site