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

!9629 stm class: Fix a double free in stm_register_device() 

Merge Pull Request from: @ci-robot 
 
PR sync from: Wang Wensheng <wangwensheng4@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/OSEFOF7UEE7QIPL2YR3TS6LYBXVKKXB6/ 
 
https://gitee.com/src-openeuler/kernel/issues/IA7D3T 
 
Link:https://gitee.com/openeuler/kernel/pulls/9629

 

Reviewed-by: default avatarWeilong Chen <chenweilong@huawei.com>
Reviewed-by: default avatarLiu YongQiang <liuyongqiang13@huawei.com>
Signed-off-by: default avatarZhang Changzhong <zhangchangzhong@huawei.com>
parents 4212574b 0ee7a0a4

drivers/hwtracing/stm/core.c

+6 −5
Original line number Diff line number Diff line
@@ -701,8 +701,11 @@ int stm_register_device(struct device *parent, struct stm_data *stm_data, 
		return -ENOMEM;



	stm->major = register_chrdev(0, stm_data->name, &stm_fops);

	if (stm->major < 0)

		goto err_free;

	if (stm->major < 0) {

		err = stm->major;

		vfree(stm);

		return err;

	}



	device_initialize(&stm->dev);

	stm->dev.devt = MKDEV(stm->major, 0);
@@ -746,10 +749,8 @@ int stm_register_device(struct device *parent, struct stm_data *stm_data, 
err_device:

	unregister_chrdev(stm->major, stm_data->name);



	/* matches device_initialize() above */

	/* calls stm_device_release() */

	put_device(&stm->dev);

err_free:

	vfree(stm);



	return err;

}