]> Pileus Git - ~andy/linux/commitdiff
md/raid10: use correct limit variable
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 11 Oct 2012 03:20:58 +0000 (14:20 +1100)
committerNeilBrown <neilb@suse.de>
Thu, 11 Oct 2012 03:20:58 +0000 (14:20 +1100)
Clang complains that we are assigning a variable to itself.  This should
be using bad_sectors like the similar earlier check does.

Bug has been present since 3.1-rc1.  It is minor but could
conceivably cause corruption or other bad behaviour.

Cc: stable@vger.kernel.org
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: NeilBrown <neilb@suse.de>
drivers/md/raid10.c

index 146749b277c6cc75dcc06ba78d9d78f81f04c2f6..867d1b4e9634682ba8184d65f4aff28dd24c3391 100644 (file)
@@ -3218,7 +3218,7 @@ static sector_t sync_request(struct mddev *mddev, sector_t sector_nr,
                                else {
                                        bad_sectors -= (sector - first_bad);
                                        if (max_sync > bad_sectors)
-                                               max_sync = max_sync;
+                                               max_sync = bad_sectors;
                                        continue;
                                }
                        }