Unverified Commit d1cbb16e authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!12161 md/raid1: don't free conf on raid0_run failure 

Merge Pull Request from: @ci-robot 
 
PR sync from: Zheng Qixing <zhengqixing@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/MYWXKXHFULIZAPM5WONN7W6QB55TCEST/ 
 
https://gitee.com/openeuler/kernel/issues/IAUQ97 
 
Link:https://gitee.com/openeuler/kernel/pulls/12161

 

Reviewed-by: default avatarYang Yingliang <yangyingliang@huawei.com>
Reviewed-by: default avatarYu Kuai <yukuai3@huawei.com>
Signed-off-by: default avatarYang Yingliang <yangyingliang@huawei.com>
parents efe8e0ec f6a28b11

drivers/md/raid1.c

+2 −10
Original line number Diff line number Diff line
@@ -3066,7 +3066,6 @@ static struct r1conf *setup_conf(struct mddev *mddev) 
	return ERR_PTR(err);

}



static void raid1_free(struct mddev *mddev, void *priv);

static int raid1_run(struct mddev *mddev)

{

	struct r1conf *conf;
@@ -3125,8 +3124,7 @@ static int raid1_run(struct mddev *mddev) 
	 */

	if (conf->raid_disks - mddev->degraded < 1) {

		md_unregister_thread(&conf->thread);

		ret = -EINVAL;

		goto abort;

		return -EINVAL;

	}



	if (conf->raid_disks - mddev->degraded == 1)
@@ -3159,14 +3157,8 @@ static int raid1_run(struct mddev *mddev) 
	}



	ret = md_integrity_register(mddev);

	if (ret) {

	if (ret)

		md_unregister_thread(&mddev->thread);

		goto abort;

	}

	return 0;



abort:

	raid1_free(mddev, conf);

	return ret;

}