From: Adrian Bunk 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 Cc: Signed-off-by: Andrew Morton --- fs/ext4/extents.c | 3 --- 1 files 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); _