From: Stefan Behrens <sbehrens@giantdisaster.de>
Date: Thu, 11 Oct 2012 13:25:16 +0000 (-0600)
Subject: Btrfs: Fix wrong error handling code
X-Git-Tag: v3.7-rc3~21^2~10
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;ds=sidebyside;h=84167d190569eedcdb24bf2499bdda437e442962;p=profile%2Fivi%2Fkernel-x86-ivi.git

Btrfs: Fix wrong error handling code

gcc says "warning: comparison of unsigned expression >= 0 is always
true" because i is an unsigned long. And gcc is right this time.

Signed-off-by: Stefan Behrens <sbehrens@giantdisaster.de>
---

diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
index 64dc93f..a32ebfe 100644
--- a/fs/btrfs/extent_io.c
+++ b/fs/btrfs/extent_io.c
@@ -4104,8 +4104,8 @@ struct extent_buffer *alloc_dummy_extent_buffer(u64 start, unsigned long len)
 
 	return eb;
 err:
-	for (i--; i >= 0; i--)
-		__free_page(eb->pages[i]);
+	for (; i > 0; i--)
+		__free_page(eb->pages[i - 1]);
 	__free_extent_buffer(eb);
 	return NULL;
 }