s390/qeth: fix early exit from error path
When the allocation of the addr buffer fails, we need to free our refcount on the inetdevice before returning. Signed-off-by: Julian Wiedmann <jwi@linux.vnet.ibm.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
025d0dfcc1
commit
83cf79a2fe
@ -1553,7 +1553,7 @@ static void qeth_l3_free_vlan_addresses4(struct qeth_card *card,
|
|||||||
|
|
||||||
addr = qeth_l3_get_addr_buffer(QETH_PROT_IPV4);
|
addr = qeth_l3_get_addr_buffer(QETH_PROT_IPV4);
|
||||||
if (!addr)
|
if (!addr)
|
||||||
return;
|
goto out;
|
||||||
|
|
||||||
spin_lock_bh(&card->ip_lock);
|
spin_lock_bh(&card->ip_lock);
|
||||||
|
|
||||||
@ -1567,6 +1567,7 @@ static void qeth_l3_free_vlan_addresses4(struct qeth_card *card,
|
|||||||
spin_unlock_bh(&card->ip_lock);
|
spin_unlock_bh(&card->ip_lock);
|
||||||
|
|
||||||
kfree(addr);
|
kfree(addr);
|
||||||
|
out:
|
||||||
in_dev_put(in_dev);
|
in_dev_put(in_dev);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -1591,7 +1592,7 @@ static void qeth_l3_free_vlan_addresses6(struct qeth_card *card,
|
|||||||
|
|
||||||
addr = qeth_l3_get_addr_buffer(QETH_PROT_IPV6);
|
addr = qeth_l3_get_addr_buffer(QETH_PROT_IPV6);
|
||||||
if (!addr)
|
if (!addr)
|
||||||
return;
|
goto out;
|
||||||
|
|
||||||
spin_lock_bh(&card->ip_lock);
|
spin_lock_bh(&card->ip_lock);
|
||||||
|
|
||||||
@ -1606,6 +1607,7 @@ static void qeth_l3_free_vlan_addresses6(struct qeth_card *card,
|
|||||||
spin_unlock_bh(&card->ip_lock);
|
spin_unlock_bh(&card->ip_lock);
|
||||||
|
|
||||||
kfree(addr);
|
kfree(addr);
|
||||||
|
out:
|
||||||
in6_dev_put(in6_dev);
|
in6_dev_put(in6_dev);
|
||||||
#endif /* CONFIG_QETH_IPV6 */
|
#endif /* CONFIG_QETH_IPV6 */
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user