Hi Marcel, On Fri, Aug 15, 2014, Marcel Holtmann wrote: > > params = hci_conn_params_lookup(hdev, &conn->dst, conn->dst_type); > > - if (params) > > + if (params) { > > list_del_init(¶ms->action); > > + if (params && params->conn) { > > don't we already know that params is valid here? Indeed we do. Silly copy-paste mistake from another piece of code where we didn't already know params was != NULL. I'll send a v2. 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