diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-01-15 09:53:52 +0300 |
|---|---|---|
| committer | Song Liu <song@kernel.org> | 2025-01-16 09:31:25 -0800 |
| commit | 62c552070a980363d55a6082b432ebd1cade7a6e (patch) | |
| tree | 8f83b6815beea7440fc1bc58b5f5376cd2d63d4c | |
| parent | 3d9a9e9a77c5ebecda43b514f2b9659644b904d0 (diff) | |
md/md-linear: Fix a NULL vs IS_ERR() bug in linear_add()
The linear_conf() returns error pointers, it doesn't return NULL. Update
the error checking to match.
Fixes: 127186cfb184 ("md: reintroduce md-linear")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Yu Kuai <yukuai3@huawei.com>
Link: https://lore.kernel.org/r/add654be-759f-4b2d-93ba-a3726dae380c@stanley.mountain
Signed-off-by: Song Liu <song@kernel.org>
| -rw-r--r-- | drivers/md/md-linear.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/md/md-linear.c b/drivers/md/md-linear.c index 53bc3fda9edb..a382929ce7ba 100644 --- a/drivers/md/md-linear.c +++ b/drivers/md/md-linear.c @@ -204,8 +204,8 @@ static int linear_add(struct mddev *mddev, struct md_rdev *rdev) rdev->saved_raid_disk = -1; newconf = linear_conf(mddev, mddev->raid_disks + 1); - if (!newconf) - return -ENOMEM; + if (IS_ERR(newconf)) + return PTR_ERR(newconf); /* newconf->raid_disks already keeps a copy of * the increased * value of mddev->raid_disks, WARN_ONCE() is just used to make |