summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohan Hedberg <johan.hedberg@intel.com>2015-06-18 21:05:31 +0300
committerJohan Hedberg <johan.hedberg@intel.com>2015-06-18 21:05:31 +0300
commit952497b159468477392f9b562b904da9bc76d468 (patch)
tree7bd7066d0ae89dbca382ed83b688c1293f0e82df
parent39ecfad68f14b71b6d1917c27eb01c4936f88bda (diff)
downloadop-kernel-dev-952497b159468477392f9b562b904da9bc76d468.zip
op-kernel-dev-952497b159468477392f9b562b904da9bc76d468.tar.gz
Bluetooth: Fix warning of potentially uninitialized adv_instance variable
Rework the logic of checking for a valid adv_instance for non-zero cp->instance values. Without this change we may get (false positive) warnings as follows: >> net/bluetooth/mgmt.c:7294:29: warning: 'adv_instance' may be used uninitialized in this function [-Wuninitialized] Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
-rw-r--r--net/bluetooth/mgmt.c8
1 files changed, 2 insertions, 6 deletions
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
index 2fe6f3b..7998fb2 100644
--- a/net/bluetooth/mgmt.c
+++ b/net/bluetooth/mgmt.c
@@ -7280,19 +7280,15 @@ static int remove_advertising(struct sock *sk, struct hci_dev *hdev,
{
struct mgmt_cp_remove_advertising *cp = data;
struct mgmt_rp_remove_advertising rp;
- struct adv_info *adv_instance;
- int err;
struct mgmt_pending_cmd *cmd;
struct hci_request req;
+ int err;
BT_DBG("%s", hdev->name);
hci_dev_lock(hdev);
- if (cp->instance)
- adv_instance = hci_find_adv_instance(hdev, cp->instance);
-
- if (!(cp->instance == 0x00 || adv_instance)) {
+ if (cp->instance && !hci_find_adv_instance(hdev, cp->instance)) {
err = mgmt_cmd_status(sk, hdev->id,
MGMT_OP_REMOVE_ADVERTISING,
MGMT_STATUS_INVALID_PARAMS);
OpenPOWER on IntegriCloud