* [PATCH v2 1/3] Bluetooth: Fix using NULL inquiry entry .
@ 2012-07-19 7:26 ramm
2012-07-19 7:26 ` [PATCH v2 2/3] Bluetooth: Fix using a NULL inquiry cache entry ramm
2012-07-19 7:26 ` [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty ramm
0 siblings, 2 replies; 6+ messages in thread
From: ramm @ 2012-07-19 7:26 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Ram Malovany
From: Ram Malovany <ramm@ti.com>
If entry wasn't found in the hci_inquiry_cache_lookup_resolve do not
resolve the name.This will fix a kernel crash when trying to use NULL
pointer.
Signed-off-by: Ram Malovany <ramm@ti.com>
---
net/bluetooth/hci_event.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
index 41ff978..2de521f 100644
--- a/net/bluetooth/hci_event.c
+++ b/net/bluetooth/hci_event.c
@@ -1365,6 +1365,9 @@ static bool hci_resolve_next_name(struct hci_dev *hdev)
return false;
e = hci_inquiry_cache_lookup_resolve(hdev, BDADDR_ANY, NAME_NEEDED);
+ if (!e)
+ return false;
+
if (hci_resolve_name(hdev, e) == 0) {
e->name_state = NAME_PENDING;
return true;
--
1.7.4.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/3] Bluetooth: Fix using a NULL inquiry cache entry.
2012-07-19 7:26 [PATCH v2 1/3] Bluetooth: Fix using NULL inquiry entry ramm
@ 2012-07-19 7:26 ` ramm
2012-07-19 7:26 ` [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty ramm
1 sibling, 0 replies; 6+ messages in thread
From: ramm @ 2012-07-19 7:26 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Ram Malovany
From: Ram Malovany <ramm@ti.com>
If the device was not found in a list of found devices names of which
are pending.This may happen in a case when HCI Remote Name Request
was sent as a part of incoming connection establishment procedure.
Hence there is no need to continue resolving a next name as it will
be done upon receiving another Remote Name Request Complete Event.
This will fix a kernel crash when trying to use this entry to resolve
the next name.
Signed-off-by: Ram Malovany <ramm@ti.com>
---
net/bluetooth/hci_event.c | 16 +++++++++++-----
1 files changed, 11 insertions(+), 5 deletions(-)
diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
index 2de521f..3224dcc 100644
--- a/net/bluetooth/hci_event.c
+++ b/net/bluetooth/hci_event.c
@@ -1396,12 +1396,18 @@ static void hci_check_pending_name(struct hci_dev *hdev, struct hci_conn *conn,
return;
e = hci_inquiry_cache_lookup_resolve(hdev, bdaddr, NAME_PENDING);
- if (e) {
+ /* If the device was not found in a list of found devices names of which
+ * are pending. there is no need to continue resolving a next name as it
+ * will be done upon receiving another Remote Name Request Complete
+ * Event */
+ if (!e)
+ return;
+
+ list_del(&e->list);
+ if (name) {
e->name_state = NAME_KNOWN;
- list_del(&e->list);
- if (name)
- mgmt_remote_name(hdev, bdaddr, ACL_LINK, 0x00,
- e->data.rssi, name, name_len);
+ mgmt_remote_name(hdev, bdaddr, ACL_LINK, 0x00,
+ e->data.rssi, name, name_len);
}
if (hci_resolve_next_name(hdev))
--
1.7.4.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty
2012-07-19 7:26 [PATCH v2 1/3] Bluetooth: Fix using NULL inquiry entry ramm
2012-07-19 7:26 ` [PATCH v2 2/3] Bluetooth: Fix using a NULL inquiry cache entry ramm
@ 2012-07-19 7:26 ` ramm
2012-07-19 14:12 ` Gustavo Padovan
1 sibling, 1 reply; 6+ messages in thread
From: ramm @ 2012-07-19 7:26 UTC (permalink / raw)
To: linux-bluetooth; +Cc: Ram Malovany
From: Ram Malovany <ramm@ti.com>
When the name of the given entry is empty , the state needs to be
updated accordingly.
Signed-off-by: Ram Malovany <ramm@ti.com>
---
net/bluetooth/hci_event.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
index 3224dcc..f799b46 100644
--- a/net/bluetooth/hci_event.c
+++ b/net/bluetooth/hci_event.c
@@ -1408,6 +1408,8 @@ static void hci_check_pending_name(struct hci_dev *hdev, struct hci_conn *conn,
e->name_state = NAME_KNOWN;
mgmt_remote_name(hdev, bdaddr, ACL_LINK, 0x00,
e->data.rssi, name, name_len);
+ } else {
+ e->name_state = NAME_NOT_KNOWN;
}
if (hci_resolve_next_name(hdev))
--
1.7.4.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty
2012-07-19 7:26 ` [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty ramm
@ 2012-07-19 14:12 ` Gustavo Padovan
2012-07-19 14:27 ` Gustavo Padovan
0 siblings, 1 reply; 6+ messages in thread
From: Gustavo Padovan @ 2012-07-19 14:12 UTC (permalink / raw)
To: ramm; +Cc: linux-bluetooth
Hi Ram,
* ramm@ti.com <ramm@ti.com> [2012-07-19 10:26:11 +0300]:
> From: Ram Malovany <ramm@ti.com>
>
> When the name of the given entry is empty , the state needs to be
> updated accordingly.
>
> Signed-off-by: Ram Malovany <ramm@ti.com>
> ---
> net/bluetooth/hci_event.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
All 3 patches have been applied to bluetooth.git. But please keep in mind that
we put periods at the end of the summary lines, I fixed that for you this
time.
Gustavo
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty
2012-07-19 14:12 ` Gustavo Padovan
@ 2012-07-19 14:27 ` Gustavo Padovan
2012-07-23 7:38 ` Malovany, Ram
0 siblings, 1 reply; 6+ messages in thread
From: Gustavo Padovan @ 2012-07-19 14:27 UTC (permalink / raw)
To: ramm, linux-bluetooth
* Gustavo Padovan <gustavo@padovan.org> [2012-07-19 11:12:05 -0300]:
> Hi Ram,
>
> * ramm@ti.com <ramm@ti.com> [2012-07-19 10:26:11 +0300]:
>
> > From: Ram Malovany <ramm@ti.com>
> >
> > When the name of the given entry is empty , the state needs to be
> > updated accordingly.
> >
> > Signed-off-by: Ram Malovany <ramm@ti.com>
> > ---
> > net/bluetooth/hci_event.c | 2 ++
> > 1 files changed, 2 insertions(+), 0 deletions(-)
>
> All 3 patches have been applied to bluetooth.git. But please keep in mind that
> we put periods at the end of the summary lines, I fixed that for you this
Sorry. We *don't* put periods at the end of summary lines.
Gustavo
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty
2012-07-19 14:27 ` Gustavo Padovan
@ 2012-07-23 7:38 ` Malovany, Ram
0 siblings, 0 replies; 6+ messages in thread
From: Malovany, Ram @ 2012-07-23 7:38 UTC (permalink / raw)
To: Gustavo Padovan, linux-bluetooth
HI Gustavo,
> -----Original Message-----
> From: Gustavo Padovan [mailto:gustavo@padovan.org]
> Sent: Thursday, July 19, 2012 5:28 PM
> To: Malovany, Ram; linux-bluetooth@vger.kernel.org
> Subject: Re: [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry
> name is empty
>
> * Gustavo Padovan <gustavo@padovan.org> [2012-07-19 11:12:05 -0300]:
>
> > Hi Ram,
> >
> > * ramm@ti.com <ramm@ti.com> [2012-07-19 10:26:11 +0300]:
> >
> > > From: Ram Malovany <ramm@ti.com>
> > >
> > > When the name of the given entry is empty , the state needs to be
> > > updated accordingly.
> > >
> > > Signed-off-by: Ram Malovany <ramm@ti.com>
> > > ---
> > > net/bluetooth/hci_event.c | 2 ++
> > > 1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > All 3 patches have been applied to bluetooth.git. But please keep in mind
> that
> > we put periods at the end of the summary lines, I fixed that for you this
>
> Sorry. We *don't* put periods at the end of summary lines.
>
> Gustavo
Thanks , I will update accordingly next time.
Regards,
Ram
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-07-23 7:38 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-19 7:26 [PATCH v2 1/3] Bluetooth: Fix using NULL inquiry entry ramm
2012-07-19 7:26 ` [PATCH v2 2/3] Bluetooth: Fix using a NULL inquiry cache entry ramm
2012-07-19 7:26 ` [PATCH v2 3/3] Bluetooth: Set name_state to unknown when entry name is empty ramm
2012-07-19 14:12 ` Gustavo Padovan
2012-07-19 14:27 ` Gustavo Padovan
2012-07-23 7:38 ` Malovany, Ram
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.