]> err.no Git - linux-2.6/commitdiff
bd_claim_by_disk: fix warning
authorAndrew Morton <akpm@linux-foundation.org>
Mon, 16 Jul 2007 06:41:28 +0000 (23:41 -0700)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Mon, 16 Jul 2007 16:05:50 +0000 (09:05 -0700)
Fix this:

fs/block_dev.c: In function 'bd_claim_by_disk':
fs/block_dev.c:970: warning: 'found' may be used uninitialized in this function

and given that free_bd_holder() now needs free(NULL)-is-legal behaviour, we
can simplify bd_release_from_kobject().

Cc: Bjorn Steinbrink <B.Steinbrink@gmx.de>
Cc: Johannes Weiner <hannes-kernel@saeurebad.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/block_dev.c

index 75c47a21b2f0507968cbcf0921fc918906fe3e68..3635315e3b99837c4c324baddb033fc2b212a99c 100644 (file)
@@ -963,20 +963,20 @@ static int bd_claim_by_kobject(struct block_device *bdev, void *holder,
 
        err = bd_claim(bdev, holder);
        if (err)
-               goto out;
+               goto fail;
 
        found = find_bd_holder(bdev, bo);
        if (found)
-               goto out;
+               goto fail;
 
        err = add_bd_holder(bdev, bo);
        if (err)
                bd_release(bdev);
-
-out:
-       if (err || found)
-               free_bd_holder(bo);
+       else
+               bo = NULL;
+fail:
        mutex_unlock(&bdev->bd_mutex);
+       free_bd_holder(bo);
        return err;
 }
 
@@ -991,15 +991,12 @@ out:
 static void bd_release_from_kobject(struct block_device *bdev,
                                        struct kobject *kobj)
 {
-       struct bd_holder *bo;
-
        if (!kobj)
                return;
 
        mutex_lock(&bdev->bd_mutex);
        bd_release(bdev);
-       if ((bo = del_bd_holder(bdev, kobj)))
-               free_bd_holder(bo);
+       free_bd_holder(del_bd_holder(bdev, kobj));
        mutex_unlock(&bdev->bd_mutex);
 }