1. 07 Dec, 2020 29 commits
  2. 23 Nov, 2020 2 commits
    • Hans de Goede's avatar
      Bluetooth: hci_h5: Add OBDA0623 ACPI HID · e524f252
      Hans de Goede authored
      Add OBDA0623 ACPI HID to the acpi_device_id table. This HID is used
      for the RTL8723BS Bluetooth part on the Acer Switch 10E SW3-016.
      
      BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1665610Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
      Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
      e524f252
    • Hans de Goede's avatar
      Bluetooth: revert: hci_h5: close serdev device and free hu in h5_close · 5c3b5796
      Hans de Goede authored
      There have been multiple revisions of the patch fix the h5->rx_skb
      leak. Accidentally the first revision (which is buggy) and v5 have
      both been merged:
      
      v1 commit 70f259a3 ("Bluetooth: hci_h5: close serdev device and free
      hu in h5_close");
      v5 commit 855af2d7 ("Bluetooth: hci_h5: fix memory leak in h5_close")
      
      The correct v5 makes changes slightly higher up in the h5_close()
      function, which allowed both versions to get merged without conflict.
      
      The changes from v1 unconditionally frees the h5 data struct, this
      is wrong because in the serdev enumeration case the memory is
      allocated in h5_serdev_probe() like this:
      
              h5 = devm_kzalloc(dev, sizeof(*h5), GFP_KERNEL);
      
      So its lifetime is tied to the lifetime of the driver being bound
      to the serdev and it is automatically freed when the driver gets
      unbound. In the serdev case the same h5 struct is re-used over
      h5_close() and h5_open() calls and thus MUST not be free-ed in
      h5_close().
      
      The serdev_device_close() added to h5_close() is incorrect in the
      same way, serdev_device_close() is called on driver unbound too and
      also MUST no be called from h5_close().
      
      This reverts the changes made by merging v1 of the patch, so that
      just the changes of the correct v5 remain.
      
      Cc: Anant Thazhemadam <anant.thazhemadam@gmail.com>
      Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
      Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
      5c3b5796
  3. 11 Nov, 2020 9 commits