Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b3f2ca94 authored by Johan Hedberg's avatar Johan Hedberg Committed by Gustavo Padovan
Browse files

Bluetooth: Fix setting local name to the existing value 



If user space attempts to set the local name to the same value that's
already set we should simply return a direct command complete for this
mgmt command.

Signed-off-by: default avatarJohan Hedberg <johan.hedberg@intel.com>
Acked-by: default avatarMarcel Holtmann <marcel@holtmann.org>
Signed-off-by: default avatarGustavo Padovan <gustavo.padovan@collabora.co.uk>
parent 3f985050

net/bluetooth/mgmt.c

+11 −0
Original line number Diff line number Diff line
@@ -2408,6 +2408,17 @@ static int set_local_name(struct sock *sk, struct hci_dev *hdev, void *data, 


	hci_dev_lock(hdev);



	/* If the old values are the same as the new ones just return a

	 * direct command complete event.

	 */

	if (!memcmp(hdev->dev_name, cp->name, sizeof(hdev->dev_name)) &&

	    !memcmp(hdev->short_name, cp->short_name,

		    sizeof(hdev->short_name))) {

		err = cmd_complete(sk, hdev->id, MGMT_OP_SET_LOCAL_NAME, 0,

				   data, len);

		goto failed;

	}



	memcpy(hdev->short_name, cp->short_name, sizeof(hdev->short_name));



	if (!hdev_is_powered(hdev)) {