Hi Marcel, On Wed, Jul 09, 2014, Marcel Holtmann wrote: > > @@ -5230,8 +5230,26 @@ static int add_device(struct sock *sk, struct hci_dev *hdev, > > MGMT_STATUS_INVALID_PARAMS, > > &cp->addr, sizeof(cp->addr)); > > > > + if (cp->action != 0x00 && cp->action != 0x01) > > + > > something is wrong here. Shouldn't this return invalid params? Yes, very good catch. Seems I did cut&paste instead of copy&paste and neither I nor the compiler noticed it. 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