]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
sh: Do not ignore INTOFF during a trap
authorJilles Tjoelker <jilles@FreeBSD.org>
Thu, 9 Jul 2020 20:53:56 +0000 (20:53 +0000)
committerJilles Tjoelker <jilles@FreeBSD.org>
Thu, 9 Jul 2020 20:53:56 +0000 (20:53 +0000)
commitbd111900918c70fb5ee5e6e5002826cee0d3b2d3
tree6cbcce75310a8030c8b837ac3ba898ccd3ab51b9
parentfe59cb6ba2e50814f0bd9f187b8baebcb648e564
sh: Do not ignore INTOFF during a trap

INTOFF postpones SIGINT processing and INTON enables it again. This is
important so an interactive shell can return to the top level prompt when
Ctrl+C is pressed.

Given that INTON is automatically done when a builtin completes, the part
where onsig() ignores suppressint when in_dotrap is true is both unnecessary
and unsafe. If the trap is for some other signal than SIGINT, arbitrary code
could have been interrupted.

Historically, INTOFF remained in effect for longer.

Reviewed by: bdrewery
MFC after: 2 weeks
Differential Revision: https://reviews.freebsd.org/D25270
bin/sh/trap.c