staging: fwserial: Fix error handling in fwserial_create
[ Upstream commit f31559af97a0eabd467e4719253675b7dccb8a46 ] When fw_core_add_address_handler() fails, we need to destroy the port by tty_port_destroy(). Also we need to unregister the address handler by fw_core_remove_address_handler() on failure. Signed-off-by: Dinghao Liu <dinghao.liu@zju.edu.cn> Link: https://lore.kernel.org/r/20201221122437.10274-1-dinghao.liu@zju.edu.cn Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
ce2b4b9c86
commit
48f6c1a894
@ -2189,6 +2189,7 @@ static int fwserial_create(struct fw_unit *unit)
|
||||
err = fw_core_add_address_handler(&port->rx_handler,
|
||||
&fw_high_memory_region);
|
||||
if (err) {
|
||||
tty_port_destroy(&port->port);
|
||||
kfree(port);
|
||||
goto free_ports;
|
||||
}
|
||||
@ -2271,6 +2272,7 @@ unregister_ttys:
|
||||
|
||||
free_ports:
|
||||
for (--i; i >= 0; --i) {
|
||||
fw_core_remove_address_handler(&serial->ports[i]->rx_handler);
|
||||
tty_port_destroy(&serial->ports[i]->port);
|
||||
kfree(serial->ports[i]);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user