From patchwork Thu Oct 6 22:49:37 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Brian Gix X-Patchwork-Id: 13000805 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id EB1DBC433FE for ; Thu, 6 Oct 2022 22:50:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231712AbiJFWuD (ORCPT ); Thu, 6 Oct 2022 18:50:03 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48988 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232101AbiJFWuC (ORCPT ); Thu, 6 Oct 2022 18:50:02 -0400 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id ABA56175A5 for ; Thu, 6 Oct 2022 15:49:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1665096600; x=1696632600; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=7bk7UDZbDxHXSF8CqN5M9PuNo3/5nKvNxXKb8WJgtDA=; b=kbhTkrW8vC1XyG+x/bHrv/6x2TsrWxggonBVvCr95U0KzWOpXq4MAqXV u5pu8fsnArSZ2Y3zZAt//S12isZ7mHGI3EdTMea3b/gaaCcAyoNfA7s2a yLsJqA1+BKEXQha3NFxdA3Uv0SuaJyU8cFTz6sJXMwq4cT5ksKEsBYzQ/ gvLwkzA7Q6xNEi8TqtwDKr0OZfnDXAL9DYTyNmAIhKq8lBfOmLcheQQhT vHdVcaypolFomRbJcMHTZPpgvlXxFv0qMzoHDs6xu4k6rfOiHWpxC6i4U dNDiW36SWGeeLOPG9OWxjPpXCl+7mALmjlLoFGQuYQjQEDzYeCivd1ZL/ w==; X-IronPort-AV: E=McAfee;i="6500,9779,10492"; a="305174716" X-IronPort-AV: E=Sophos;i="5.95,164,1661842800"; d="scan'208";a="305174716" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Oct 2022 15:49:58 -0700 X-IronPort-AV: E=McAfee;i="6500,9779,10492"; a="750327029" X-IronPort-AV: E=Sophos;i="5.95,164,1661842800"; d="scan'208";a="750327029" Received: from ctamayo-mobl2.amr.corp.intel.com (HELO bgi1-mobl2.amr.corp.intel.com) ([10.212.51.159]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Oct 2022 15:49:48 -0700 From: Brian Gix To: linux-bluetooth@vger.kernel.org Cc: luiz.dentz@gmail.com, brian.gix@intel.com Subject: [PATCH] Bluetooth: Call HCI cmd to set random addr during MGMT call Date: Thu, 6 Oct 2022 15:49:37 -0700 Message-Id: <20221006224937.641179-1-brian.gix@intel.com> X-Mailer: git-send-email 2.37.3 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org The call to MGMT_OP_SET_STATIC_ADDRESS saved the requested address in the hdev structure, but never wrote that address out to the controller. This adds call to hci_set_random_addr_sync() after it has been validated. Signed-off-by: Brian Gix --- include/net/bluetooth/hci_sync.h | 1 + net/bluetooth/hci_sync.c | 2 +- net/bluetooth/mgmt.c | 35 ++++++++++++++++++++++++++++++++ 3 files changed, 37 insertions(+), 1 deletion(-) diff --git a/include/net/bluetooth/hci_sync.h b/include/net/bluetooth/hci_sync.h index 17f5a4c32f36..24864672d12c 100644 --- a/include/net/bluetooth/hci_sync.h +++ b/include/net/bluetooth/hci_sync.h @@ -90,6 +90,7 @@ int hci_read_clock_sync(struct hci_dev *hdev, struct hci_cp_read_clock *cp); int hci_write_fast_connectable_sync(struct hci_dev *hdev, bool enable); int hci_update_scan_sync(struct hci_dev *hdev); int hci_update_scan(struct hci_dev *hdev); +int hci_set_random_addr_sync(struct hci_dev *hdev, bdaddr_t *rpa); int hci_write_le_host_supported_sync(struct hci_dev *hdev, u8 le, u8 simul); int hci_remove_ext_adv_instance_sync(struct hci_dev *hdev, u8 instance, diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c index 422f7c6911d9..8fd16ddd015a 100644 --- a/net/bluetooth/hci_sync.c +++ b/net/bluetooth/hci_sync.c @@ -839,7 +839,7 @@ static bool adv_use_rpa(struct hci_dev *hdev, uint32_t flags) return true; } -static int hci_set_random_addr_sync(struct hci_dev *hdev, bdaddr_t *rpa) +int hci_set_random_addr_sync(struct hci_dev *hdev, bdaddr_t *rpa) { /* If we're advertising or initiating an LE connection we can't * go ahead and change the random address at this time. This is diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index a92e7e485feb..aff861e0e655 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -6501,10 +6501,28 @@ static int set_advertising(struct sock *sk, struct hci_dev *hdev, void *data, return err; } +static int set_addr_sync(struct hci_dev *hdev, void *data) +{ + struct mgmt_pending_cmd *cmd = data; + struct mgmt_cp_set_static_address *cp = cmd->param; + + return hci_set_random_addr_sync(hdev, &cp->bdaddr); +} + +static void set_addr_complete(struct hci_dev *hdev, void *data, int err) +{ + struct mgmt_pending_cmd *cmd = data; + + mgmt_cmd_status(cmd->sk, hdev->id, MGMT_OP_SET_STATIC_ADDRESS, + mgmt_status(err)); + mgmt_pending_remove(cmd); +} + static int set_static_address(struct sock *sk, struct hci_dev *hdev, void *data, u16 len) { struct mgmt_cp_set_static_address *cp = data; + struct mgmt_pending_cmd *cmd = data; int err; bt_dev_dbg(hdev, "sock %p", sk); @@ -6534,6 +6552,23 @@ static int set_static_address(struct sock *sk, struct hci_dev *hdev, bacpy(&hdev->static_addr, &cp->bdaddr); + cmd = mgmt_pending_add(sk, MGMT_OP_SET_STATIC_ADDRESS, hdev, data, len); + if (!cmd) + err = -ENOMEM; + else + err = hci_cmd_sync_queue(hdev, set_addr_sync, cmd, + set_addr_complete); + + if (err < 0) { + if (cmd) + mgmt_pending_remove(cmd); + + mgmt_cmd_status(sk, hdev->id, MGMT_OP_SET_STATIC_ADDRESS, + mgmt_status(err)); + + goto unlock; + } + err = send_settings_rsp(sk, MGMT_OP_SET_STATIC_ADDRESS, hdev); if (err < 0) goto unlock;