diff options
| author | Lexi Winter <lexi@le-Fay.ORG> | 2024-01-04 22:34:58 +0000 |
|---|---|---|
| committer | Mark Johnston <markj@FreeBSD.org> | 2024-01-05 02:40:53 +0000 |
| commit | 6c951b37170f1fb2ae8b4827070743e61b6eaed2 (patch) | |
| tree | d5e2e933a9bcd677edecfb1f0bf908713902c5be /secure | |
| parent | 324fd7ec40439e6b3916429a69956d7acf74eb19 (diff) | |
| download | src-6c951b37170f1fb2ae8b4827070743e61b6eaed2.tar.gz src-6c951b37170f1fb2ae8b4827070743e61b6eaed2.zip | |
mail: add volatile in grabh()
setjmp() requires that any stack variables modified between the setjmp
call and the longjmp() must be volatile. This means that 'saveint' in
grabh() must be volatile, since it's modified after the setjmp().
Otherwise, the signal handler is not properly restored, resulting in a
crash (SIGBUS) if ^C is typed twice while composing.
PR: 276119
Reported by: Christopher Davidson <christopher.davidson@gmail.com>
MFC after: 2 weeks
Pull Request: https://github.com/freebsd/freebsd-src/pull/993
Diffstat (limited to 'secure')
0 files changed, 0 insertions, 0 deletions