diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-10-26 03:42:36 -0400 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2011-10-26 03:42:36 -0400 |
| commit | f85b287a01237857a50c93868231f7e831581a27 (patch) | |
| tree | 370cc4ce183e6321192ddd60b2a1e056932a235d /fs/ext4 | |
| parent | 665436175c3ca9d35f135e1ba6bdd63745cff08a (diff) | |
| download | linux-stable-f85b287a01237857a50c93868231f7e831581a27.tar.gz linux-stable-f85b287a01237857a50c93868231f7e831581a27.tar.bz2 linux-stable-f85b287a01237857a50c93868231f7e831581a27.zip | |
ext4: error handling fix in ext4_ext_convert_to_initialized()
When allocated is unsigned it breaks the error handling at the end
of the function when we call:
allocated = ext4_split_extent(...);
if (allocated < 0)
err = allocated;
I've made it a signed int instead of unsigned.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
| -rw-r--r-- | fs/ext4/extents.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index 8686eb756b38..02a4d80573a1 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -2928,7 +2928,8 @@ static int ext4_ext_convert_to_initialized(handle_t *handle, struct ext4_extent zero_ex; struct ext4_extent *ex; ext4_lblk_t ee_block, eof_block; - unsigned int allocated, ee_len, depth; + unsigned int ee_len, depth; + int allocated; int err = 0; int split_flag = 0; |