Hi, On Thu, Jul 21, 2011, mike tsai wrote: > Hi, > This patch reads the discovered characteristic's property to > determine if Read is permitted before sending read request. This > avoids the error response from server as "read not permitted". > > --- > attrib/client.c | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/attrib/client.c b/attrib/client.c > index 1ef2699..f345a9e 100644 > --- a/attrib/client.c > +++ b/attrib/client.c > @@ -850,6 +850,8 @@ static void update_all_chars(gpointer data, > gpointer user_data) > struct primary *prim = user_data; > struct gatt_service *gatt = prim->gatt; > > + if (chr->perm & ATT_CHAR_PROPER_READ) > + { > qdesc = g_new0(struct query_data, 1); > qdesc->prim = prim; > qdesc->chr = chr; > @@ -864,8 +866,8 @@ static void update_all_chars(gpointer data, > gpointer user_data) > > gatt->attrib = g_attrib_ref(gatt->attrib); > gatt_read_char(gatt->attrib, chr->handle, 0, update_char_value, qvalue); > + } > } > - > static void char_discovered_cb(GSList *characteristics, guint8 status, Besides the obvious coding-style issue, is this patch still valid? Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html