Hi Johan, On Thu, Nov 11, 2010 at 7:18 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > Hi Claudio, > > On Wed, Nov 10, 2010, Claudio Takahasi wrote: >> @@ -623,6 +624,9 @@ int attrib_server_init(void) >> >> Â Â Â sdp_handle = record->handle; >> >> + Â Â if (!main_opts.le) >> + Â Â Â Â Â Â return 0; >> + > > If the option is called "le" it shouldn't affect GATT over BR/EDR, so I > think the check in attrib_server_init should only affect the LE socket > (or then rename the option to e.g. main.opts.attrib). > > Johan > It is not affecting GATT over BR/EDR. bt_io_listen for BR/EDR is above this checking. My patch "[PATCH] Fix possible memory leak of the GIOChannel in the attribute server" switched the order of the bt_io_listen calls between BR/EDR and LE. I forgot to send these patches as patch sequence. Claudio. -- 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
