On Sun, 2012-06-17 at 22:58 +0200, Lekensteyn wrote: > Before this patch, reading log_event would print the event log to syslog and > crash if CONFIG_IWLWIFI_DEBUG was disabled (while CONFIG_IWLWIFI_DEBUGFS was > enabled of course). The crash was caused by (1) iwl_dbgfs_log_event_read failing > to initialize variable buf, thereby causing the following kfree(buf) to be > executed. (2) iwl_dump_nic_event_log does not initialize buf if > CONFIG_IWLWIFI_DEBUG was disabled. I've applied a different fix for this, just disabling the log_event file when CONFIG_IWLWIFI_DEBUG is disabled. This looks like it could use some disentangling but I'm not sure what you did is really the right approach, some refactoring would seem to be better. Also, is "Lekensteyn" really your full name? Not that it matters now because I didn't apply your patch, but for patch submissions please read the "developer's certificate of origin" in the kernel sources. Thanks for the report! johannes -- 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
