lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 31 Oct 2014 10:17:45 +0100 From: Paul Bolle <pebolle@...cali.nl> To: Johannes Berg <johannes@...solutions.net> Cc: Valentin Rothberg <valentinrothberg@...il.com>, Emmanuel Grumbach <emmanuel.grumbach@...el.com>, "John W. Linville" <linville@...driver.com>, Intel Linux Wireless <ilw@...ux.intel.com>, linux-wireless@...r.kernel.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: iwlwifi: mvm: BACKPORT_WANT_DEV_COREDUMP? On Fri, 2014-10-31 at 10:08 +0100, Johannes Berg wrote: > On Fri, 2014-10-31 at 10:06 +0100, Paul Bolle wrote: > > Perhaps you could also look into somehow guarding the call of > > dev_coredumpm(), that this commit added, with checks for > > CONFIG_DEV_COREDUMP. See, I had a quick look at all this and selecting > > WANT_DEV_COREDUMP might not be enough, because DISABLE_DEV_COREDUMP can > > still, well, disable DEV_COREDUMP. Or am I misreading the Kconfig > > symbols that regulate DEV_COREDUMP? > > No, you're correctly reading that. However, the devcoredump header file > provides simple functions in this case. That means there's some extra > work (allocating and filling the buffer just to free it immediately) but > it simplifies the code. I see. More than a quick look was required here. Thanks for explaining this! Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists