udf: Truncate added extents on failed expansion

commit 70bfb3a8d661d4fdc742afc061b88a7f3fc9f500 upstream.

When a file expansion failed because we didn't have enough space for
indirect extents make sure we truncate extents created so far so that we
don't leave extents beyond EOF.

CC: stable@vger.kernel.org
Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/fs/udf/inode.c b/fs/udf/inode.c
index e92a16435a29e7d4cdafb1d4913e0b908f1fe07e..f8908c1f482eab341a38ca9db37f17def1f8959a 100644 (file)
--- a/fs/udf/inode.c
+++ b/fs/udf/inode.c
@@ -526,8 +526,10 @@ static int udf_do_extend_file(struct inode *inode,
        }
 
        if (fake) {
-               udf_add_aext(inode, last_pos, &last_ext->extLocation,
-                            last_ext->extLength, 1);
+               err = udf_add_aext(inode, last_pos, &last_ext->extLocation,
+                                  last_ext->extLength, 1);
+               if (err < 0)
+                       goto out_err;
                count++;
        } else {
                struct kernel_lb_addr tmploc;
@@ -561,7 +563,7 @@ static int udf_do_extend_file(struct inode *inode,
                err = udf_add_aext(inode, last_pos, &last_ext->extLocation,
                                   last_ext->extLength, 1);
                if (err)
-                       return err;
+                       goto out_err;
                count++;
        }
        if (new_block_bytes) {
@@ -570,7 +572,7 @@ static int udf_do_extend_file(struct inode *inode,
                err = udf_add_aext(inode, last_pos, &last_ext->extLocation,
                                   last_ext->extLength, 1);
                if (err)
-                       return err;
+                       goto out_err;
                count++;
        }
 
@@ -584,6 +586,11 @@ out:
                return -EIO;
 
        return count;
+out_err:
+       /* Remove extents we've created so far */
+       udf_clear_extent_cache(inode);
+       udf_truncate_extents(inode);
+       return err;
 }
 
 /* Extend the final block of the file to final_block_len bytes */