Hi Vishal, On Wed, Apr 04, 2012, Vishal Agarwal wrote: > @@ -1330,10 +1330,10 @@ int hci_add_link_key(struct hci_dev *hdev, struct hci_conn *conn, int new_key, > > mgmt_new_link_key(hdev, key, persistent); > > - if (!persistent) { > - list_del(&key->list); > - kfree(key); > - } > + if (persistent) > + conn->temp_link_key = false; > + else > + conn->temp_link_key = true; The hci_add_link_key function can be called with conn == NULL so you need to take this into account before dereferencing 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