Bluetooth: mgmt: Remove unwanted goto statements
authorSyam Sidhardhan <s.syam@samsung.com>
Thu, 12 Apr 2012 15:03:21 +0000 (20:33 +0530)
committerGustavo Padovan <gustavo@padovan.org>
Wed, 9 May 2012 04:40:33 +0000 (01:40 -0300)
Remove goto statements that do nothing else than jump to the next line
of code.

Signed-off-by: Syam Sidhardhan <s.syam@samsung.com>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
net/bluetooth/mgmt.c

index 5e80c6db6ae75d3a2bd1278bb3f168dc73f57740..b50e2ee2595cde996965c19f523dcdf9084d9f7f 100644 (file)
@@ -1247,10 +1247,8 @@ static int set_le(struct sock *sk, struct hci_dev *hdev, void *data, u16 len)
 
        err = hci_send_cmd(hdev, HCI_OP_WRITE_LE_HOST_SUPPORTED, sizeof(hci_cp),
                           &hci_cp);
-       if (err < 0) {
+       if (err < 0)
                mgmt_pending_remove(cmd);
-               goto unlock;
-       }
 
 unlock:
        hci_dev_unlock(hdev);
@@ -1300,10 +1298,8 @@ static int add_uuid(struct sock *sk, struct hci_dev *hdev, void *data, u16 len)
        }
 
        cmd = mgmt_pending_add(sk, MGMT_OP_ADD_UUID, hdev, data, len);
-       if (!cmd) {
+       if (!cmd)
                err = -ENOMEM;
-               goto failed;
-       }
 
 failed:
        hci_dev_unlock(hdev);
@@ -1388,10 +1384,8 @@ update_class:
        }
 
        cmd = mgmt_pending_add(sk, MGMT_OP_REMOVE_UUID, hdev, data, len);
-       if (!cmd) {
+       if (!cmd)
                err = -ENOMEM;
-               goto unlock;
-       }
 
 unlock:
        hci_dev_unlock(hdev);
@@ -1442,10 +1436,8 @@ static int set_dev_class(struct sock *sk, struct hci_dev *hdev, void *data,
        }
 
        cmd = mgmt_pending_add(sk, MGMT_OP_SET_DEV_CLASS, hdev, data, len);
-       if (!cmd) {
+       if (!cmd)
                err = -ENOMEM;
-               goto unlock;
-       }
 
 unlock:
        hci_dev_unlock(hdev);