diff options
author | Theodore Ts'o <tytso@mit.edu> | 2008-12-06 16:58:39 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2008-12-06 16:58:39 -0500 |
commit | 59e315b4c410b00a9acd0f24a00dbadbe81ce692 (patch) | |
tree | d19e7b45d98c42c6f94cb5aa292e2efea44244f5 /fs/ext3/namei.c | |
parent | f99b25897a86fcfff9140396a97261ae65fed872 (diff) |
ext3/4: Fix loop index in do_split() so it is signed
This fixes a gcc warning but it doesn't appear able to result in a
failure, since the primary way the loop is exited is the first
conditional in the for loop, and at least for a consistent filesystem,
the signed/unsigned should in practice never be exposed.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext3/namei.c')
-rw-r--r-- | fs/ext3/namei.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/ext3/namei.c b/fs/ext3/namei.c index 287b304d42a7..2c2d700c1ccf 100644 --- a/fs/ext3/namei.c +++ b/fs/ext3/namei.c | |||
@@ -1161,9 +1161,9 @@ static struct ext3_dir_entry_2 *do_split(handle_t *handle, struct inode *dir, | |||
1161 | u32 hash2; | 1161 | u32 hash2; |
1162 | struct dx_map_entry *map; | 1162 | struct dx_map_entry *map; |
1163 | char *data1 = (*bh)->b_data, *data2; | 1163 | char *data1 = (*bh)->b_data, *data2; |
1164 | unsigned split, move, size, i; | 1164 | unsigned split, move, size; |
1165 | struct ext3_dir_entry_2 *de = NULL, *de2; | 1165 | struct ext3_dir_entry_2 *de = NULL, *de2; |
1166 | int err = 0; | 1166 | int err = 0, i; |
1167 | 1167 | ||
1168 | bh2 = ext3_append (handle, dir, &newblock, &err); | 1168 | bh2 = ext3_append (handle, dir, &newblock, &err); |
1169 | if (!(bh2)) { | 1169 | if (!(bh2)) { |