Hi, > On 04/17/09 02:18, Chris Wright wrote: >> * Ivo van Doorn (ivdoorn@xxxxxxxxx) wrote: >>> At rmmod stage, the code path is the following one : >>> >>> rt2x00lib_remove_dev >>> -> rt2x00lib_uninitialize() >>> -> rt2x00rfkill_unregister() >>> -> rfkill_unregister() >>> -> rt2x00rfkill_free() >>> -> rfkill_free() >>> >>> The problem is that rfkill_free should not be called after rfkill_register >>> otherwise put_device(&rfkill->dev) will be called 2 times. This patch >>> fixes this by only calling rt2x00rfkill_free() when rt2x00rfkill_register() >>> hasn't been called or has failed. >>> >>> Signed-off-by: Gertjan van Wingerde <gwingerde@xxxxxxxxx> >>> Tested-by: Arnaud Patard <apatard@xxxxxxxxxxxx> >>> Signed-off-by: Ivo van Doorn <IvDoorn@xxxxxxxxx> >> >> Who is the Author of this patch? The Signed-off-by order suggests is >> Gertjan. >> > > > Indeed, I have been the author of the patch. Yeah, Gertjan is indeed the author, I forgot to add the Fom: line at the top of the patch to make that more clear. Ivo -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
