Hi Johan, On Wed, Jan 9, 2013 at 9:29 AM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > @@ -1219,6 +1247,10 @@ static int set_le(struct sock *sk, struct hci_dev *hdev, void *data, u16 len) > return cmd_status(sk, hdev->id, MGMT_OP_SET_LE, > MGMT_STATUS_NOT_SUPPORTED); > > + if (cp->val != 0x00 && cp->val != 0x01) > + return cmd_status(sk, hdev->id, MGMT_OP_SET_SSP, > + MGMT_STATUS_INVALID_PARAMS); > + I think you meant "MGMT_OP_SET_LE" here. Can mgmt-tester be modified to check for this as well? Regards, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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