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

!9017 misc/libmasm/module: Fix two use after free in ibmasm_init_one 

Merge Pull Request from: @ci-robot 
 
PR sync from: Zhang Changzhong <zhangchangzhong@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/FUJCPDZDBPC5Q46ZSEHJQBZI6GZX43YH/ 
 
https://gitee.com/src-openeuler/kernel/issues/I9RLTD 
 
Link:https://gitee.com/openeuler/kernel/pulls/9017

 

Reviewed-by: default avatarLiu YongQiang <liuyongqiang13@huawei.com>
Signed-off-by: default avatarZhang Changzhong <zhangchangzhong@huawei.com>
parents 0b15078d 83226016

drivers/misc/ibmasm/module.c

+3 −2
Original line number Diff line number Diff line
@@ -125,7 +125,7 @@ static int ibmasm_init_one(struct pci_dev *pdev, const struct pci_device_id *id) 
	result = ibmasm_init_remote_input_dev(sp);

	if (result) {

		dev_err(sp->dev, "Failed to initialize remote queue\n");

		goto error_send_message;

		goto error_init_remote;

	}



	result = ibmasm_send_driver_vpd(sp);
@@ -145,8 +145,9 @@ static int ibmasm_init_one(struct pci_dev *pdev, const struct pci_device_id *id) 
	return 0;



error_send_message:

	disable_sp_interrupts(sp->base_address);

	ibmasm_free_remote_input_dev(sp);

error_init_remote:

	disable_sp_interrupts(sp->base_address);

	free_irq(sp->irq, (void *)sp);

error_request_irq:

	iounmap(sp->base_address);