Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-3128-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed,  3 Apr 2024 19:05:10 +0800
Precedence: bulk
Mime-Version: 1.0
Message-ID: <20240403110549.75516-1-howardchung@google.com>
Subject: [Bluez PATCH 0/2] Fix descriptor display issue in btmon
From: Howard Chung <howardchung@google.com>
To: linux-bluetooth <linux-bluetooth@vger.kernel.org>, 
	Luiz Augusto von Dentz <luiz.dentz@gmail.com>, Johan Hedberg <johan.hedberg@gmail.com>, 
	Marcel Holtmann <marcel@holtmann.org>
Cc: CrosBT Upstreaming <chromeos-bluetooth-upstreaming@chromium.org>, 
	Howard Chung <howardchung@google.com>
Content-Type: text/plain; charset="UTF-8"

Hi Maintainers, we found that in some cases the new feature regarding
displaying the characterisitc of the descriptor could be wrong. It turns
out it is because there is an assumption (please see the 2nd commit) in
the gatt-db usage but not being properly maintained in monitor/att.c.
These patches are aimming to fix the issue. I'd be happy to send you the
btsnoop log but not sure what's the best way for you to send it to you.
Thanks.


Howard Chung (2):
  shared/gatt: Rename some gatt insert functions to append
  shared/gatt: Add descriptor insert function

 monitor/att.c            |   2 +-
 src/gatt-database.c      |   4 +-
 src/settings.c           |   4 +-
 src/shared/gatt-client.c |   8 +--
 src/shared/gatt-db.c     | 102 ++++++++++++++++++++++++++++++++-------
 src/shared/gatt-db.h     |  19 ++++++--
 unit/test-gatt.c         |   4 +-
 7 files changed, 109 insertions(+), 34 deletions(-)

-- 
2.44.0.478.gd926399ef9-goog