From: Dan Carpenter <error27@gmail.com>
Date: Mon, 8 Mar 2010 12:15:59 +0000 (+0300)
Subject: Bluetooth: Fix storing negative values as unsigned char
X-Git-Tag: firefly_0821_release~9833^2~2087^2~175
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=31f085fea891d5972e994dfb423231a3d4d60a8a;p=firefly-linux-kernel-4.4.55.git

Bluetooth: Fix storing negative values as unsigned char

This is mostly cleanup. There is only one caller and it just checks for
non-zero return values. Still "ret" should be int because we want to return
-EINVAL on errors.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
---

diff --git a/drivers/bluetooth/btmrvl_main.c b/drivers/bluetooth/btmrvl_main.c
index 53a43adf2e21..49ec89cd354c 100644
--- a/drivers/bluetooth/btmrvl_main.c
+++ b/drivers/bluetooth/btmrvl_main.c
@@ -66,7 +66,7 @@ int btmrvl_process_event(struct btmrvl_private *priv, struct sk_buff *skb)
 {
 	struct btmrvl_adapter *adapter = priv->adapter;
 	struct btmrvl_event *event;
-	u8 ret = 0;
+	int ret = 0;
 
 	event = (struct btmrvl_event *) skb->data;
 	if (event->ec != 0xff) {