lkml.org 
[lkml]   [2015]   [Feb]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.18 13/57] udf: Release preallocation on last writeable close
    Date
    3.18-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Jan Kara <jack@suse.cz>

    commit b07ef35244424cbeda9844198607c7077099c82c upstream.

    Commit 6fb1ca92a640 "udf: Fix race between write(2) and close(2)"
    changed the condition when preallocation is released. The idea was that
    we don't want to release the preallocation for an inode on close when
    there are other writeable file descriptors for the inode. However the
    condition was written in the opposite way so we released preallocation
    only if there were other writeable file descriptors. Fix the problem by
    changing the condition properly.

    Fixes: 6fb1ca92a6409a9d5b0696447cd4997bc9aaf5a2
    Reported-by: Fabian Frederick <fabf@skynet.be>
    Signed-off-by: Jan Kara <jack@suse.cz>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    fs/udf/file.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    --- a/fs/udf/file.c
    +++ b/fs/udf/file.c
    @@ -224,7 +224,7 @@ out:
    static int udf_release_file(struct inode *inode, struct file *filp)
    {
    if (filp->f_mode & FMODE_WRITE &&
    - atomic_read(&inode->i_writecount) > 1) {
    + atomic_read(&inode->i_writecount) == 1) {
    /*
    * Grab i_mutex to avoid races with writes changing i_size
    * while we are running.



    \
     
     \ /
      Last update: 2015-02-04 01:21    [W:2.686 / U:0.272 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site