Merge branch 'for-upstream' of git://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth
Johan Hedberg says: ==================== Here are a couple of important Bluetooth & mac802154 fixes for 4.1: - mac802154 fix for crypto algorithm allocation failure checking - mac802154 wpan phy leak fix for error code path - Fix for not calling Bluetooth shutdown() if interface is not up Let me know if there are any issues pulling. Thanks. ==================== Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
commit
0a801445db
3 changed files with 7 additions and 4 deletions
|
@ -1557,7 +1557,8 @@ static int hci_dev_do_close(struct hci_dev *hdev)
|
|||
{
|
||||
BT_DBG("%s %p", hdev->name, hdev);
|
||||
|
||||
if (!hci_dev_test_flag(hdev, HCI_UNREGISTER)) {
|
||||
if (!hci_dev_test_flag(hdev, HCI_UNREGISTER) &&
|
||||
test_bit(HCI_UP, &hdev->flags)) {
|
||||
/* Execute vendor specific shutdown routine */
|
||||
if (hdev->shutdown)
|
||||
hdev->shutdown(hdev);
|
||||
|
|
|
@ -134,7 +134,7 @@ llsec_key_alloc(const struct ieee802154_llsec_key *template)
|
|||
for (i = 0; i < ARRAY_SIZE(key->tfm); i++) {
|
||||
key->tfm[i] = crypto_alloc_aead("ccm(aes)", 0,
|
||||
CRYPTO_ALG_ASYNC);
|
||||
if (!key->tfm[i])
|
||||
if (IS_ERR(key->tfm[i]))
|
||||
goto err_tfm;
|
||||
if (crypto_aead_setkey(key->tfm[i], template->key,
|
||||
IEEE802154_LLSEC_KEY_SIZE))
|
||||
|
@ -144,7 +144,7 @@ llsec_key_alloc(const struct ieee802154_llsec_key *template)
|
|||
}
|
||||
|
||||
key->tfm0 = crypto_alloc_blkcipher("ctr(aes)", 0, CRYPTO_ALG_ASYNC);
|
||||
if (!key->tfm0)
|
||||
if (IS_ERR(key->tfm0))
|
||||
goto err_tfm;
|
||||
|
||||
if (crypto_blkcipher_setkey(key->tfm0, template->key,
|
||||
|
|
|
@ -167,13 +167,15 @@ int ieee802154_register_hw(struct ieee802154_hw *hw)
|
|||
if (IS_ERR(dev)) {
|
||||
rtnl_unlock();
|
||||
rc = PTR_ERR(dev);
|
||||
goto out_wq;
|
||||
goto out_phy;
|
||||
}
|
||||
|
||||
rtnl_unlock();
|
||||
|
||||
return 0;
|
||||
|
||||
out_phy:
|
||||
wpan_phy_unregister(local->phy);
|
||||
out_wq:
|
||||
destroy_workqueue(local->workqueue);
|
||||
out:
|
||||
|
|
Loading…
Reference in a new issue