udf: Propagate errors from udf_truncate_extents()
authorJan Kara <jack@suse.cz>
Mon, 11 Mar 2019 14:27:02 +0000 (15:27 +0100)
committerJan Kara <jack@suse.cz>
Mon, 18 Mar 2019 15:30:02 +0000 (16:30 +0100)
Make udf_truncate_extents() properly propagate errors to its callers and
let udf_setsize() handle the error properly as well. This lets userspace
know in case there's some error when truncating blocks.

Signed-off-by: Jan Kara <jack@suse.cz>
fs/udf/inode.c
fs/udf/truncate.c
fs/udf/udfdecl.h

index ae796e10f68b2524423dae6d3840c7a9d05cf4c3..e7276932e433c9cdc1bdd10f9a35a3d0ef09ece7 100644 (file)
@@ -1242,8 +1242,10 @@ set_size:
                truncate_setsize(inode, newsize);
                down_write(&iinfo->i_data_sem);
                udf_clear_extent_cache(inode);
-               udf_truncate_extents(inode);
+               err = udf_truncate_extents(inode);
                up_write(&iinfo->i_data_sem);
+               if (err)
+                       return err;
        }
 update_time:
        inode->i_mtime = inode->i_ctime = current_time(inode);
index 94220ba8562858e7fa30dfef8734da506b24dfa0..63a47f1e1d529c1c98a1f25c86cd25880c1737ac 100644 (file)
@@ -199,7 +199,7 @@ static void udf_update_alloc_ext_desc(struct inode *inode,
  * for making file shorter. For making file longer, udf_extend_file() has to
  * be used.
  */
-void udf_truncate_extents(struct inode *inode)
+int udf_truncate_extents(struct inode *inode)
 {
        struct extent_position epos;
        struct kernel_lb_addr eloc, neloc = {};
@@ -224,7 +224,7 @@ void udf_truncate_extents(struct inode *inode)
        if (etype == -1) {
                /* We should extend the file? */
                WARN_ON(byte_offset);
-               return;
+               return 0;
        }
        epos.offset -= adsize;
        extent_trunc(inode, &epos, &eloc, etype, elen, byte_offset);
@@ -262,7 +262,7 @@ void udf_truncate_extents(struct inode *inode)
                                        udf_get_lb_pblock(sb, &eloc, 0));
                        /* Error reading indirect block? */
                        if (!epos.bh)
-                               return;
+                               return -EIO;
                        if (elen)
                                indirect_ext_len =
                                        (elen + sb->s_blocksize - 1) >>
@@ -286,4 +286,5 @@ void udf_truncate_extents(struct inode *inode)
        iinfo->i_lenExtents = inode->i_size;
 
        brelse(epos.bh);
+       return 0;
 }
index ee246769dee4a2c54f31b56c5237f77f7db3a040..d89ef71887fcfa1aed13bacc77a1c42ee3a55c1a 100644 (file)
@@ -235,7 +235,7 @@ extern struct inode *udf_new_inode(struct inode *, umode_t);
 /* truncate.c */
 extern void udf_truncate_tail_extent(struct inode *);
 extern void udf_discard_prealloc(struct inode *);
-extern void udf_truncate_extents(struct inode *);
+extern int udf_truncate_extents(struct inode *);
 
 /* balloc.c */
 extern void udf_free_blocks(struct super_block *, struct inode *,