diff options
| author | Johan Hedberg <johan.hedberg@intel.com> | 2014-03-27 13:51:24 +0200 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2014-03-27 09:00:58 -0700 |
| commit | 8002d77ca9edbf81b81ab5154d75f7cce3d0511e (patch) | |
| tree | 68ded614732e68d421ff1c7cf00c5a1159f79afb | |
| parent | ab0aa433e2f6c69e69b4d5a951c0b84e7b193f0d (diff) | |
| download | op-kernel-dev-8002d77ca9edbf81b81ab5154d75f7cce3d0511e.zip op-kernel-dev-8002d77ca9edbf81b81ab5154d75f7cce3d0511e.tar.gz | |
Bluetooth: Remove unnecessary NULL check in hci_inquiry_cache_update
The ssp parameter is supposed to be a mandatory one and there are no
callers that would pass NULL to this function. Removing this unnecessary
NULL check also makes (false positive) static analyzer warnings go away.
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
| -rw-r--r-- | net/bluetooth/hci_core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c index c6189d9..7cf511c 100644 --- a/net/bluetooth/hci_core.c +++ b/net/bluetooth/hci_core.c @@ -2032,7 +2032,7 @@ bool hci_inquiry_cache_update(struct hci_dev *hdev, struct inquiry_data *data, ie = hci_inquiry_cache_lookup(hdev, &data->bdaddr); if (ie) { - if (ie->data.ssp_mode && ssp) + if (ie->data.ssp_mode) *ssp = true; if (ie->name_state == NAME_NEEDED && |
