diff options
| author |   Hans de Goede <hdegoede@redhat.com> | 2018-05-27 21:04:51 +0200 |
|---|---|---|
| committer |   Marcel Holtmann <marcel@holtmann.org> | 2018-05-30 08:46:17 +0200 |
| commit | 412fe29ffcf9a1ba9f125e72ee3da957cd6a0064 (patch) | |
| tree | 24e60ae102719943d15a3030ef34fef681b8be02 /drivers/bluetooth/hci_serdev.c | |
| parent | Bluetooth: btmrvl: Re-use kstrtol_from_user() (diff) | |
| download | linux-dev-412fe29ffcf9a1ba9f125e72ee3da957cd6a0064.tar.xz linux-dev-412fe29ffcf9a1ba9f125e72ee3da957cd6a0064.zip | |
Bluetooth: hci_uart: Restore hci_dev->flush callback on open()
For reasons explained in detail in commit 3611f4d2a5e0 ("hci_ldisc:
fix null pointer deref") the hci_uart_close() functions sets
hci_dev->flush to NULL.
But the device may be re-opened after a close, this commit restores the
hci_dev->flush callback on open().
Note this commit also moves the nearly empty defition of hci_uart_open()
a bit down in the file to avoid the need for forward declaring
hci_uart_flush().
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'drivers/bluetooth/hci_serdev.c')
| -rw-r--r-- | drivers/bluetooth/hci_serdev.c | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/drivers/bluetooth/hci_serdev.c b/drivers/bluetooth/hci_serdev.c index a0e939856568..6a713f13f71c 100644 --- a/drivers/bluetooth/hci_serdev.c +++ b/drivers/bluetooth/hci_serdev.c @@ -101,14 +101,6 @@ static void hci_uart_write_work(struct work_struct *work) /* ------- Interface to HCI layer ------ */ -/* Initialize device */ -static int hci_uart_open(struct hci_dev *hdev) -{ - BT_DBG("%s %p", hdev->name, hdev); - - return 0; -} - /* Reset device */ static int hci_uart_flush(struct hci_dev *hdev) { @@ -129,6 +121,17 @@ static int hci_uart_flush(struct hci_dev *hdev) return 0; } +/* Initialize device */ +static int hci_uart_open(struct hci_dev *hdev) +{ + BT_DBG("%s %p", hdev->name, hdev); + + /* Undo clearing this from hci_uart_close() */ + hdev->flush = hci_uart_flush; + + return 0; +} + /* Close device */ static int hci_uart_close(struct hci_dev *hdev) { |