From: Archie Pusaka <[email protected]>
According to bluetooth spec Ver 5.2, Vol 3, Part G, 7.1, the
service_changed characteristic is not readable. Therefore, this
patch marks it as such.
---
src/gatt-database.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gatt-database.c b/src/gatt-database.c
index f2da27694..8cbe09bae 100644
--- a/src/gatt-database.c
+++ b/src/gatt-database.c
@@ -1197,7 +1197,7 @@ static void populate_gatt_service(struct btd_gatt_database *database)
bt_uuid16_create(&uuid, GATT_CHARAC_SERVICE_CHANGED);
database->svc_chngd = gatt_db_service_add_characteristic(service, &uuid,
- BT_ATT_PERM_READ, BT_GATT_CHRC_PROP_INDICATE,
+ BT_ATT_PERM_NONE, BT_GATT_CHRC_PROP_INDICATE,
NULL, NULL, database);
database->svc_chngd_ccc = service_add_ccc(service, database, NULL, NULL,
--
2.26.1.301.g55bc3eb7cb9-goog
Hi BlueZ maintainers,
Please kindly take a look at this patch, thank you!
On Tue, 21 Apr 2020 at 15:59, Archie Pusaka <[email protected]> wrote:
>
> From: Archie Pusaka <[email protected]>
>
> According to bluetooth spec Ver 5.2, Vol 3, Part G, 7.1, the
> service_changed characteristic is not readable. Therefore, this
> patch marks it as such.
> ---
>
> src/gatt-database.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gatt-database.c b/src/gatt-database.c
> index f2da27694..8cbe09bae 100644
> --- a/src/gatt-database.c
> +++ b/src/gatt-database.c
> @@ -1197,7 +1197,7 @@ static void populate_gatt_service(struct btd_gatt_database *database)
>
> bt_uuid16_create(&uuid, GATT_CHARAC_SERVICE_CHANGED);
> database->svc_chngd = gatt_db_service_add_characteristic(service, &uuid,
> - BT_ATT_PERM_READ, BT_GATT_CHRC_PROP_INDICATE,
> + BT_ATT_PERM_NONE, BT_GATT_CHRC_PROP_INDICATE,
> NULL, NULL, database);
>
> database->svc_chngd_ccc = service_add_ccc(service, database, NULL, NULL,
> --
> 2.26.1.301.g55bc3eb7cb9-goog
>
Hi Archie,
On Mon, Apr 27, 2020 at 9:31 PM Archie Pusaka <[email protected]> wrote:
>
> Hi BlueZ maintainers,
>
> Please kindly take a look at this patch, thank you!
>
> On Tue, 21 Apr 2020 at 15:59, Archie Pusaka <[email protected]> wrote:
> >
> > From: Archie Pusaka <[email protected]>
> >
> > According to bluetooth spec Ver 5.2, Vol 3, Part G, 7.1, the
> > service_changed characteristic is not readable. Therefore, this
> > patch marks it as such.
> > ---
> >
> > src/gatt-database.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/gatt-database.c b/src/gatt-database.c
> > index f2da27694..8cbe09bae 100644
> > --- a/src/gatt-database.c
> > +++ b/src/gatt-database.c
> > @@ -1197,7 +1197,7 @@ static void populate_gatt_service(struct btd_gatt_database *database)
> >
> > bt_uuid16_create(&uuid, GATT_CHARAC_SERVICE_CHANGED);
> > database->svc_chngd = gatt_db_service_add_characteristic(service, &uuid,
> > - BT_ATT_PERM_READ, BT_GATT_CHRC_PROP_INDICATE,
> > + BT_ATT_PERM_NONE, BT_GATT_CHRC_PROP_INDICATE,
> > NULL, NULL, database);
> >
> > database->svc_chngd_ccc = service_add_ccc(service, database, NULL, NULL,
> > --
> > 2.26.1.301.g55bc3eb7cb9-goog
Applied, thanks.
--
Luiz Augusto von Dentz