From: Adrian Bunk <bunk@xxxxxxxxx> The Coverity checker noted that this was dead code, since in all places above in this function, "err" is immediately checked. Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx> Cc: <linux-ext4@xxxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxx> --- fs/ext4/extents.c | 3 --- 1 file changed, 3 deletions(-) diff -puN fs/ext4/extents.c~ext4_ext_split-remove-dead-code fs/ext4/extents.c --- a/fs/ext4/extents.c~ext4_ext_split-remove-dead-code +++ a/fs/ext4/extents.c @@ -800,9 +800,6 @@ static int ext4_ext_split(handle_t *hand } /* insert new index */ - if (err) - goto cleanup; - err = ext4_ext_insert_index(handle, inode, path + at, le32_to_cpu(border), newblock); _ - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
