Hi Frédéric, On Thu, May 31, 2012 at 11:30 AM, Frédéric Dalleau <frederic.dalleau@xxxxxxxxxxxxxxx> wrote: > - if (req->type == BNEP_CONTROL && > + if ((req->type & ~BNEP_EXT_HEADER) == BNEP_CONTROL && > req->ctrl > BNEP_FILTER_MULT_ADDR_RSP) { > uint8_t pkt[3]; > > @@ -390,7 +416,8 @@ static gboolean bnep_setup(GIOChannel *chan, > return FALSE; > } > > - if (req->type != BNEP_CONTROL || req->ctrl != BNEP_SETUP_CONN_REQ) > + if ((req->type & ~BNEP_EXT_HEADER) != BNEP_CONTROL || > + req->ctrl != BNEP_SETUP_CONN_REQ) > return FALSE; > Perhaps you can have a macro for getting just the header type without the extension bit e.g. BNEP_HEADER(type) ((type) & ~BNEP_EXT_HEADER) -- Luiz Augusto von Dentz -- 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