Commit 61f45415 authored Jun 23, 2022 by Jakub Pawlowski Committed by Jakub Pawłowski Jun 24, 2022

GATT: Unregister notifications when disconnecting

Currently, if we receive notification after disconnection, but before
closing the client, stack would notify upper layers that connection is
avaliable, and send notifications to Java.

This can be troublesome i.e. for Heart Rate Monitors, that send
notifications very often, and can prevent apps from closing properly.

Bug: 231818024
Test: connect to HRM with more than one application, close/disconnect
multiple times

Change-Id: Icb68c9107bb9dbd54ca0e48d4cabea5da143055e

parent 1762aaf8

system/bta/gatt/bta_gattc_act.cc

+8 −0

Original line number	Diff line number	Diff line
		@@ -588,6 +588,14 @@ void bta_gattc_close(tBTA_GATTC_CLCB* p_clcb, const tBTA_GATTC_DATA* p_data) {
		if (p_clcb->transport == BT_TRANSPORT_BR_EDR)
		bta_sys_conn_close(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda);

		/* Clean up notification registration for closed client */
		for (int i = 0; i < BTA_GATTC_NOTIF_REG_MAX; i++) {
		if (p_clreg->notif_reg[i].in_use &&
		p_clreg->notif_reg[i].remote_bda == p_clcb->bda) {
		p_clreg->notif_reg[i] = {};
		}
		}

		bta_gattc_clcb_dealloc(p_clcb);

		if (p_data->hdr.event == BTA_GATTC_API_CLOSE_EVT) {