Hi Lizardo, On Tue, Feb 22, 2011, Anderson Lizardo wrote: > +static uint8_t client_set_notifications(struct attribute *attr, > + gpointer user_data) > +{ > + struct gatt_channel *channel = user_data; > + struct attribute *a, *last_chr_val = NULL; > + uint16_t handle, cfg_val; > + uuid_t uuid; > + GSList *l; > + > + cfg_val = att_get_u16(attr->data); > + > + sdp_uuid16_create(&uuid, GATT_CHARAC_UUID); > + for (l = database, handle = 0; l != NULL; l = l->next) { > + a = l->data; The variable "a" is only used inside the for-loop so it should be declared inside it as well. I think you can move handle inside the loop as well as long as you declare it static (so it only gets initialized to 0 on the first iteration). 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