Hi Johan, On Wed, Apr 27, 2011 at 7:57 PM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > Hi Lizardo, > > On Wed, Apr 27, 2011, Anderson Lizardo wrote: >> > + if (new_key && !hci_persistent_key(hdev, conn, type, old_key_type)) { >> > + list_del(&key->list); >> > + kfree(key); >> > + return 0; >> >> Just to make sure, is this condition possible? >> >> * new_key parameter not zero >> * old_key variable not NULL >> >> If so, "key" will point to an entry from hdev->link_keys , which will >> be freed on the if() above without removing it from the list. > > Do you see the list_del call there? :) Ok, I failed to see it :) 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