Commit 68d740d7 authored by Silva Paulo's avatar Silva Paulo Committed by Linus Torvalds

blk: fix wrong idr_pre_get() error check in loop.c

The idr_pre_get() function never returns a value < 0.  It returns 0 (no
memory) or 1 (OK).
Reported-by: default avatarSilva Paulo <psdasilva@yahoo.com>
[ Rewrote Silva's patch, but attributing it to Silva anyway  - Linus ]
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 1daaa5e4
...@@ -1597,14 +1597,12 @@ static int loop_add(struct loop_device **l, int i) ...@@ -1597,14 +1597,12 @@ static int loop_add(struct loop_device **l, int i)
struct gendisk *disk; struct gendisk *disk;
int err; int err;
err = -ENOMEM;
lo = kzalloc(sizeof(*lo), GFP_KERNEL); lo = kzalloc(sizeof(*lo), GFP_KERNEL);
if (!lo) { if (!lo)
err = -ENOMEM;
goto out; goto out;
}
err = idr_pre_get(&loop_index_idr, GFP_KERNEL); if (!idr_pre_get(&loop_index_idr, GFP_KERNEL))
if (err < 0)
goto out_free_dev; goto out_free_dev;
if (i >= 0) { if (i >= 0) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment