diff mbox series

[Bluez] shared/gatt-client: fix BT_ATT_OP_HANDLE_NFY_MULT parsing

Message ID 20210316121212.Bluez.1.I92177a7c526b6d3e04c59f95d846ddfa79977b51@changeid
State New
Headers show
Series [Bluez] shared/gatt-client: fix BT_ATT_OP_HANDLE_NFY_MULT parsing | expand

Commit Message

Archie Pusaka March 16, 2021, 4:12 a.m. UTC 
From: Archie Pusaka <apusaka@chromium.org>

We need to advance the pdu pointer after parsing each block of
notification data.

Reviewed-by: Sonny Sasaka <sonnysasaka@chromium.org>
---

 src/shared/gatt-client.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Luiz Augusto von Dentz March 16, 2021, 6:07 p.m. UTC | #1 
Hi Archie,

On Tue, Mar 16, 2021 at 1:05 AM <bluez.test.bot@gmail.com> wrote:
>
> This is automated email and please do not reply to this email!
>
> Dear submitter,
>
> Thank you for submitting the patches to the linux bluetooth mailing list.
> This is a CI test results with your patch series:
> PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=448757
>
> ---Test result---
>
> ##############################
> Test: CheckPatch - PASS
>
> ##############################
> Test: CheckGitLint - PASS
>
> ##############################
> Test: CheckBuild - PASS
>
> ##############################
> Test: MakeCheck - PASS
>
>
>
> ---
> Regards,
> Linux Bluetooth

Applied, thanks.
diff mbox series

Patch

diff --git a/src/shared/gatt-client.c b/src/shared/gatt-client.c
index 8becf1c6cd..4e10f244d8 100644
--- a/src/shared/gatt-client.c
+++ b/src/shared/gatt-client.c
@@ -2179,6 +2179,7 @@  static void notify_cb(struct bt_att_chan *chan, uint8_t opcode,
 								&data);
 
 			length -= data.len;
+			pdu += data.len;
 		}
 	} else {
 		data.handle = get_le16(pdu);