Hi Luiz, On Fri, Jun 10, 2011 at 2:50 AM, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote: > diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c > index 3163330..efda44c 100644 > --- a/net/bluetooth/hci_conn.c > +++ b/net/bluetooth/hci_conn.c > @@ -611,8 +611,8 @@ auth: > if (test_and_set_bit(HCI_CONN_ENCRYPT_PEND, &conn->pend)) > return 0; > > - hci_conn_auth(conn, sec_level, auth_type); > - return 0; > + if (!(hci_conn_auth(conn, sec_level, auth_type))) > + return 0; Too much nitpicking, but why the extra parenthesis? 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
