diff options
| author | Matthias Kaehlcke <mka@chromium.org> | 2018-09-20 10:12:53 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-11-27 11:16:35 +0100 |
| commit | 8341f2f222d729688014ce8306727fdb9798d37e (patch) | |
| tree | b1cab8b656ee5361cabb7a61327681b6c23aec88 /crypto | |
| parent | 279070b96a5a087ba712e6ff333fa126d3e6fbb7 (diff) | |
| download | lwn-8341f2f222d729688014ce8306727fdb9798d37e.tar.gz lwn-8341f2f222d729688014ce8306727fdb9798d37e.zip | |
sysrq: Use panic() to force a crash
sysrq_handle_crash() currently forces a crash by dereferencing a
NULL pointer, which is undefined behavior in C. Just call panic()
instead, which is simpler and doesn't depend on compiler specific
handling of the undefined behavior.
Remove the comment on why the RCU lock needs to be released, it isn't
accurate anymore since the crash now isn't handled by the page fault
handler (for reference: the comment was added by commit 984cf355aeaa
("sysrq: Fix warning in sysrq generated crash.")). Releasing the lock
is still good practice though.
Suggested-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'crypto')
0 files changed, 0 insertions, 0 deletions