diff options
| author | Christoph Hellwig <hch@tuxera.com> | 2011-02-02 17:40:33 +0100 |
|---|---|---|
| committer | Christoph Hellwig <hch@tuxera.com> | 2011-02-04 00:34:18 +0100 |
| commit | 1065348d472f97b4b8eb53b60ec67e99148cbbca (patch) | |
| tree | a8724027733f5991f90951d6491cf5946e853325 /fs/hfsplus/extents.c | |
| parent | hfsplus: fix two memory leaks in wrapper.c (diff) | |
| download | linux-1065348d472f97b4b8eb53b60ec67e99148cbbca.tar.xz linux-1065348d472f97b4b8eb53b60ec67e99148cbbca.zip | |
hfsplus: fix up a comparism in hfsplus_file_extend
Revert an incorrect hunk from commit b2837fcf4994e699a4def002e26f274d95b387c1,
"hfsplus: %L-to-%ll, macro correction, and remove unneeded braces"
revert a pointless change of comparism operation argument order, which turned
out to not even be equivalent.
Reported-by: Joe Perches <joe@perches.com>
Signed-off-by: Christoph Hellwig <hch@tuxera.com>
Diffstat (limited to 'fs/hfsplus/extents.c')
| -rw-r--r-- | fs/hfsplus/extents.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/hfsplus/extents.c b/fs/hfsplus/extents.c index 52a0bcaa7b6d..b1991a2a08e0 100644 --- a/fs/hfsplus/extents.c +++ b/fs/hfsplus/extents.c @@ -397,8 +397,8 @@ int hfsplus_file_extend(struct inode *inode) u32 start, len, goal; int res; - if (sbi->total_blocks - sbi->free_blocks + 8 > - sbi->alloc_file->i_size * 8) { + if (sbi->alloc_file->i_size * 8 < + sbi->total_blocks - sbi->free_blocks + 8) { /* extend alloc file */ printk(KERN_ERR "hfs: extend alloc file! " "(%llu,%u,%u)\n", |