* [PATCH] Bluetooth: Fix return value when bt_skb_alloc fails
@ 2010-04-22 6:20 Gustavo F. Padovan
2010-04-22 6:54 ` Gustavo F. Padovan
0 siblings, 1 reply; 2+ messages in thread
From: Gustavo F. Padovan @ 2010-04-22 6:20 UTC (permalink / raw)
To: linux-bluetooth; +Cc: gustavo, marcel, Gustavo F. Padovan
From: Gustavo F. Padovan <padovan@profusion.mobi>
Set the proper error(ENOMEM), instead of just return 0.
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
---
drivers/bluetooth/hci_h4.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/bluetooth/hci_h4.c b/drivers/bluetooth/hci_h4.c
index c0ce813..3f038f5 100644
--- a/drivers/bluetooth/hci_h4.c
+++ b/drivers/bluetooth/hci_h4.c
@@ -246,7 +246,7 @@ static int h4_recv(struct hci_uart *hu, void *data, int count)
BT_ERR("Can't allocate mem for new packet");
h4->rx_state = H4_W4_PACKET_TYPE;
h4->rx_count = 0;
- return 0;
+ return -ENOMEM;
}
h4->rx_skb->dev = (void *) hu->hdev;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Bluetooth: Fix return value when bt_skb_alloc fails
2010-04-22 6:20 [PATCH] Bluetooth: Fix return value when bt_skb_alloc fails Gustavo F. Padovan
@ 2010-04-22 6:54 ` Gustavo F. Padovan
0 siblings, 0 replies; 2+ messages in thread
From: Gustavo F. Padovan @ 2010-04-22 6:54 UTC (permalink / raw)
To: linux-bluetooth; +Cc: marcel
Hi Marcel,
* Gustavo F. Padovan <gustavo@padovan.org> [2010-04-22 03:20:30 -0300]:
> From: Gustavo F. Padovan <padovan@profusion.mobi>
>
> Set the proper error(ENOMEM), instead of just return 0.
>
> Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
> ---
> drivers/bluetooth/hci_h4.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/bluetooth/hci_h4.c b/drivers/bluetooth/hci_h4.c
> index c0ce813..3f038f5 100644
> --- a/drivers/bluetooth/hci_h4.c
> +++ b/drivers/bluetooth/hci_h4.c
> @@ -246,7 +246,7 @@ static int h4_recv(struct hci_uart *hu, void *data, int count)
> BT_ERR("Can't allocate mem for new packet");
> h4->rx_state = H4_W4_PACKET_TYPE;
> h4->rx_count = 0;
> - return 0;
> + return -ENOMEM;
> }
>
> h4->rx_skb->dev = (void *) hu->hdev;
> --
> 1.7.0.4
>
Ignore this one, I have found a similar issue in other file, just sent
the new fix to the list.
--
Gustavo F. Padovan
http://padovan.org
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-04-22 6:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-04-22 6:20 [PATCH] Bluetooth: Fix return value when bt_skb_alloc fails Gustavo F. Padovan
2010-04-22 6:54 ` Gustavo F. Padovan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).