Bluetooth: hci_uart_set_flow_control: Fix NULL deref when using serdev
authorHans de Goede <hdegoede@redhat.com>
Wed, 4 Oct 2017 18:43:35 +0000 (20:43 +0200)
committerMarcel Holtmann <marcel@holtmann.org>
Fri, 6 Oct 2017 18:35:47 +0000 (20:35 +0200)
Fix a NULL pointer deref (hu->tty) when calling hci_uart_set_flow_control
on hci_uart-s using serdev.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/hci_ldisc.c

index a746627e784e7a55f895e1e98c1be47467bdb136..eec95019f15c819d3b6a5f46b107e160ccfd1216 100644 (file)
@@ -41,6 +41,7 @@
 #include <linux/ioctl.h>
 #include <linux/skbuff.h>
 #include <linux/firmware.h>
+#include <linux/serdev.h>
 
 #include <net/bluetooth/bluetooth.h>
 #include <net/bluetooth/hci_core.h>
@@ -298,6 +299,12 @@ void hci_uart_set_flow_control(struct hci_uart *hu, bool enable)
        unsigned int set = 0;
        unsigned int clear = 0;
 
+       if (hu->serdev) {
+               serdev_device_set_flow_control(hu->serdev, !enable);
+               serdev_device_set_rts(hu->serdev, !enable);
+               return;
+       }
+
        if (enable) {
                /* Disable hardware flow control */
                ktermios = tty->termios;