[PATCH] Bluetooth: Route traffic only through BR/EDR controller

[PATCH] Bluetooth: Route traffic only through BR/EDR controller

[Andrei Emeltchenko]

From: Andrei Emeltchenko <andrei.emeltchenko@intel.com>

If AMP controller is first in the list then Bluetooth traffic might
be routed through it (if source is not specified). The patch
prevents this case and also checks that source is BR/EDR.

Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
---
 net/bluetooth/hci_conn.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
index 9aac198..3ec5fc2 100644
--- a/net/bluetooth/hci_conn.c
+++ b/net/bluetooth/hci_conn.c
@@ -442,7 +442,8 @@ struct hci_dev *hci_get_route(bdaddr_t *dst, bdaddr_t *src)
 
 	list_for_each_entry(d, &hci_dev_list, list) {
 		if (!test_bit(HCI_UP, &d->flags) ||
-		    test_bit(HCI_RAW, &d->flags))
+		    test_bit(HCI_RAW, &d->flags) ||
+		    d->dev_type != HCI_BREDR)
 			continue;
 
 		/* Simple routing:
-- 
1.7.9.5

Re: [PATCH] Bluetooth: Route traffic only through BR/EDR controller

[Andrei Emeltchenko]

On Tue, Jun 19, 2012 at 03:21:21PM +0300, Andrei Emeltchenko wrote:
> From: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
> 
> If AMP controller is first in the list then Bluetooth traffic might
> be routed through it (if source is not specified). The patch
> prevents this case and also checks that source is BR/EDR.
> 
> Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>

ping


> ---
>  net/bluetooth/hci_conn.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
> index 9aac198..3ec5fc2 100644
> --- a/net/bluetooth/hci_conn.c
> +++ b/net/bluetooth/hci_conn.c
> @@ -442,7 +442,8 @@ struct hci_dev *hci_get_route(bdaddr_t *dst, bdaddr_t *src)
>  
>  	list_for_each_entry(d, &hci_dev_list, list) {
>  		if (!test_bit(HCI_UP, &d->flags) ||
> -		    test_bit(HCI_RAW, &d->flags))
> +		    test_bit(HCI_RAW, &d->flags) ||
> +		    d->dev_type != HCI_BREDR)
>  			continue;
>  
>  		/* Simple routing:
> -- 
> 1.7.9.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Bluetooth: Route traffic only through BR/EDR controller

[Gustavo Padovan]

Hi Andrei,

* Andrei Emeltchenko <Andrei.Emeltchenko.news@gmail.com> [2012-06-19 15:21:21 +0300]:

> From: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
> 
> If AMP controller is first in the list then Bluetooth traffic might
> be routed through it (if source is not specified). The patch
> prevents this case and also checks that source is BR/EDR.
> 
> Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
> ---
>  net/bluetooth/hci_conn.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Patch has been applied to bluetooth-next, thanks.

	Gustavo