Patch "Bluetooth: btintel: Fix missing free skb in btintel_setup_combined()" has been added to the 5.15-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    Bluetooth: btintel: Fix missing free skb in btintel_setup_combined()

to the 5.15-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     bluetooth-btintel-fix-missing-free-skb-in-btintel_se.patch
and it can be found in the queue-5.15 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 533cbfc1f0590d2e74b3aa47adbbeefb3ec17eaa
Author: Wang ShaoBo <bobo.shaobowang@xxxxxxxxxx>
Date:   Mon Dec 5 21:53:57 2022 +0800

    Bluetooth: btintel: Fix missing free skb in btintel_setup_combined()
    
    [ Upstream commit cee50ce899de415baf4da3ed38b7d4f13c3170d1 ]
    
    skb allocated by __hci_cmd_sync would not be used whether in checking
    for supported iBT hardware variants or after, we should free it in all
    error branches, this patch makes the case read version failed or default
    error case free skb before return.
    
    Fixes: c86c7285bb08 ("Bluetooth: btintel: Fix the legacy bootloader returns tlv based version")
    Fixes: 019a1caa7fd2 ("Bluetooth: btintel: Refactoring setup routine for bootloader devices")
    Signed-off-by: Wang ShaoBo <bobo.shaobowang@xxxxxxxxxx>
    Reviewed-by: Tedd Ho-Jeong An <tedd.an@xxxxxxxxx>
    Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c
index de3d851d85e7..d707aa63e944 100644
--- a/drivers/bluetooth/btintel.c
+++ b/drivers/bluetooth/btintel.c
@@ -2353,7 +2353,7 @@ static int btintel_setup_combined(struct hci_dev *hdev)
 		 */
 		err = btintel_read_version(hdev, &ver);
 		if (err)
-			return err;
+			break;
 
 		/* Apply the device specific HCI quirks
 		 *
@@ -2394,7 +2394,8 @@ static int btintel_setup_combined(struct hci_dev *hdev)
 	default:
 		bt_dev_err(hdev, "Unsupported Intel hw variant (%u)",
 			   INTEL_HW_VARIANT(ver_tlv.cnvi_bt));
-		return -EINVAL;
+		err = -EINVAL;
+		break;
 	}
 
 exit_error:



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux