diff options
author | Christoph Hellwig <hch@tuxera.com> | 2011-02-02 09:40:33 -0700 |
---|---|---|
committer | Christoph Hellwig <hch@tuxera.com> | 2011-02-03 16:34:18 -0700 |
commit | 1065348d472f97b4b8eb53b60ec67e99148cbbca (patch) | |
tree | a8724027733f5991f90951d6491cf5946e853325 /fs/hfsplus | |
parent | a1dbcef0172555464b5329f8ba47d43c98132dfa (diff) | |
download | lwn-1065348d472f97b4b8eb53b60ec67e99148cbbca.tar.gz lwn-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')
-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", |