[PATCH] Bluetooth: mt7921s: fix a NULL check

[PATCH] Bluetooth: mt7921s: fix a NULL check

[Dan Carpenter]

There is a typo in the NULL check so it's never true.  It should be
checking "*ven_data" instead of "ven_data".

Fixes: 3cabc5ca2c9d ("Bluetooth: mt7921s: Add .btmtk_get_codec_config_data")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
 drivers/bluetooth/btmtksdio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/bluetooth/btmtksdio.c b/drivers/bluetooth/btmtksdio.c
index f3dc5881fff7..b6d77e04240c 100644
--- a/drivers/bluetooth/btmtksdio.c
+++ b/drivers/bluetooth/btmtksdio.c
@@ -961,7 +961,7 @@ static int btmtksdio_get_codec_config_data(struct hci_dev *hdev,
 	}
 
 	*ven_data = kmalloc(sizeof(__u8), GFP_KERNEL);
-	if (!ven_data) {
+	if (!*ven_data) {
 		err = -ENOMEM;
 		goto error;
 	}
-- 
2.20.1

回复: [PATCH] Bluetooth: mt7921s: fix a NULL check

[Yake Yang (杨亚克)]

Hi Dan,
Thanks for fixing this issue, we will update patch.

-----邮件原件-----
发件人: Dan Carpenter <dan.carpenter@oracle.com> 
发送时间: 2022年3月23日 15:49
收件人: Marcel Holtmann <marcel@holtmann.org>; Yake Yang (杨亚克) <yake.yang@mediatek.com>
抄送: Johan Hedberg <johan.hedberg@gmail.com>; Luiz Augusto von Dentz <luiz.dentz@gmail.com>; Matthias Brugger <matthias.bgg@gmail.com>; Sean Wang <Sean.Wang@mediatek.com>; Mark Chen <markyawenchen@gmail.com>; linux-bluetooth@vger.kernel.org; linux-mediatek@lists.infradead.org; kernel-janitors@vger.kernel.org
主题: [PATCH] Bluetooth: mt7921s: fix a NULL check

There is a typo in the NULL check so it's never true.  It should be checking "*ven_data" instead of "ven_data".

Fixes: 3cabc5ca2c9d ("Bluetooth: mt7921s: Add .btmtk_get_codec_config_data")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
 drivers/bluetooth/btmtksdio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/bluetooth/btmtksdio.c b/drivers/bluetooth/btmtksdio.c index f3dc5881fff7..b6d77e04240c 100644
--- a/drivers/bluetooth/btmtksdio.c
+++ b/drivers/bluetooth/btmtksdio.c
@@ -961,7 +961,7 @@ static int btmtksdio_get_codec_config_data(struct hci_dev *hdev,
 	}
 
 	*ven_data = kmalloc(sizeof(__u8), GFP_KERNEL);
-	if (!ven_data) {
+	if (!*ven_data) {
 		err = -ENOMEM;
 		goto error;
 	}
--
2.20.1

RE: 回复: [PATCH] Bluetooth: mt7921s: fix a NULL check

[bluez.test.bot]

[-- Attachment #1: Type: text/plain, Size: 509 bytes --]

This is an automated email and please do not reply to this email.

Dear Submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
While preparing the CI tests, the patches you submitted couldn't be applied to the current HEAD of the repository.

----- Output -----
error: cannot convert from eucgb2312_cn to UTF-8
error: cannot convert from eucgb2312_cn to UTF-8
fatal: could not parse patch


Please resolve the issue and submit the patches again.


---
Regards,
Linux Bluetooth

RE: Bluetooth: mt7921s: fix a NULL check

[bluez.test.bot]

[-- Attachment #1: Type: text/plain, Size: 560 bytes --]

This is an automated email and please do not reply to this email.

Dear Submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
While preparing the CI tests, the patches you submitted couldn't be applied to the current HEAD of the repository.

----- Output -----
error: patch failed: drivers/bluetooth/btmtksdio.c:961
error: drivers/bluetooth/btmtksdio.c: patch does not apply
hint: Use 'git am --show-current-patch' to see the failed patch


Please resolve the issue and submit the patches again.


---
Regards,
Linux Bluetooth

Re: Bluetooth: mt7921s: fix a NULL check

[Dan Carpenter]

On Wed, Mar 23, 2022 at 01:49:26AM -0700, bluez.test.bot@gmail.com wrote:
> This is an automated email and please do not reply to this email.
> 
> Dear Submitter,
> 
> Thank you for submitting the patches to the linux bluetooth mailing list.
> While preparing the CI tests, the patches you submitted couldn't be applied to the current HEAD of the repository.
> 
> ----- Output -----
> error: patch failed: drivers/bluetooth/btmtksdio.c:961
> error: drivers/bluetooth/btmtksdio.c: patch does not apply
> hint: Use 'git am --show-current-patch' to see the failed patch
> 
> 
> Please resolve the issue and submit the patches again.

I tested it and it applies to:

git://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master

It would help if the email said what tree the build-bot is using.  Who
is maintaining the build-bot?

regards,
dan carpenter

Re: [PATCH] Bluetooth: mt7921s: fix a NULL check

[patchwork-bot+bluetooth]

Hello:

This patch was applied to bluetooth/bluetooth-next.git (master)
by Marcel Holtmann <marcel@holtmann.org>:

On Wed, 23 Mar 2022 10:48:30 +0300 you wrote:
> There is a typo in the NULL check so it's never true.  It should be
> checking "*ven_data" instead of "ven_data".
> 
> Fixes: 3cabc5ca2c9d ("Bluetooth: mt7921s: Add .btmtk_get_codec_config_data")
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> ---
>  drivers/bluetooth/btmtksdio.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Here is the summary with links:
  - Bluetooth: mt7921s: fix a NULL check
    https://git.kernel.org/bluetooth/bluetooth-next/c/864cc8a234cd

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html