Return-Path: Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 7.2 \(1874\)) Subject: Re: [PATCH v2] Bluetooth: Fix response on confirm_name From: Marcel Holtmann In-Reply-To: <1393513976-5497-1-git-send-email-lukasz.rymanowski@tieto.com> Date: Thu, 27 Feb 2014 07:47:31 -0800 Cc: linux-bluetooth@vger.kernel.org Message-Id: <9A316920-E118-4B8B-B494-0B0396887144@holtmann.org> References: <1393513976-5497-1-git-send-email-lukasz.rymanowski@tieto.com> To: Lukasz Rymanowski Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Lukasz, > According to mgmt-api.txt, in case of confirm name command, > cmd_complete should be always use as a response. Not command status > as it is now for failures. > Using command complete on failure is actually better as client might > be interested in device address for which confirm name failed. > > Signed-off-by: Lukasz Rymanowski > --- > net/bluetooth/mgmt.c | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > index 4c4912e..8b7a3c4 100644 > --- a/net/bluetooth/mgmt.c > +++ b/net/bluetooth/mgmt.c > @@ -3627,15 +3627,17 @@ static int confirm_name(struct sock *sk, struct hci_dev *hdev, void *data, > hci_dev_lock(hdev); > > if (!hci_discovery_active(hdev)) { > - err = cmd_status(sk, hdev->id, MGMT_OP_CONFIRM_NAME, > - MGMT_STATUS_FAILED); > + err = cmd_complete(sk, hdev->id, MGMT_OP_CONFIRM_NAME, > + MGMT_STATUS_FAILED, &cp->addr, > + sizeof(cp->addr)); you are testing me, right ;) Regards Marcel