mirrors/foot
1
0
Fork 0
mirror of https://codeberg.org/dnkl/foot.git synced 2026-03-16 05:34:00 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge branch 'fdm-always-check-for-signals'

Browse source 
Closes #534
This commit is contained in:
Daniel Eklöf 2021-05-22 14:47:44 +02:00
commit af428d544f
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F
2 changed files with 22 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

3
CHANGELOG.md
View file

@ -110,6 +110,9 @@
scaling factor > 1 (https://codeberg.org/dnkl/foot/issues/509). scaling factor > 1 (https://codeberg.org/dnkl/foot/issues/509).
* Crash caused by certain CSI sequences with very large parameter * Crash caused by certain CSI sequences with very large parameter
values (https://codeberg.org/dnkl/foot/issues/522). values (https://codeberg.org/dnkl/foot/issues/522).
* Rare occurrences where the window did not close when the shell
exited. Only seen on FreeBSD
(https://codeberg.org/dnkl/foot/issues/534)
### Security ### Security

34
fdm.c
View file

@ -52,6 +52,7 @@ struct fdm {
hooks_t hooks_high; hooks_t hooks_high;
}; };
static volatile sig_atomic_t got_signal = false;
static volatile sig_atomic_t *received_signals = NULL; static volatile sig_atomic_t *received_signals = NULL;
struct fdm * struct fdm *
@ -71,6 +72,7 @@ fdm_init(void)
xassert(received_signals == NULL); /* Only one FDM instance supported */ xassert(received_signals == NULL); /* Only one FDM instance supported */
received_signals = xcalloc(SIGRTMAX, sizeof(received_signals[0])); received_signals = xcalloc(SIGRTMAX, sizeof(received_signals[0]));
got_signal = false;
struct fdm *fdm = malloc(sizeof(*fdm)); struct fdm *fdm = malloc(sizeof(*fdm));
if (unlikely(fdm == NULL)) { if (unlikely(fdm == NULL)) {
@ -328,6 +330,7 @@ fdm_hook_del(struct fdm *fdm, fdm_hook_t hook, enum fdm_hook_priority priority)
static void static void
signal_handler(int signo) signal_handler(int signo)
{ {
got_signal = true;
received_signals[signo] = true; received_signals[signo] = true;
} }
@ -424,27 +427,28 @@ fdm_poll(struct fdm *fdm)
int r = epoll_pwait( int r = epoll_pwait(
fdm->epoll_fd, events, tll_length(fdm->fds), -1, &fdm->sigmask); fdm->epoll_fd, events, tll_length(fdm->fds), -1, &fdm->sigmask);
if (unlikely(r < 0)) { int errno_copy = errno;
if (errno == EINTR) {
/* TODO: is it possible to receive a signal without
* getting EINTR here? */
for (int i = 0; i < SIGRTMAX; i++) { if (unlikely(got_signal)) {
if (received_signals[i]) { got_signal = false;
received_signals[i] = false; for (int i = 0; i < SIGRTMAX; i++) {
struct sig_handler *handler = &fdm->signal_handlers[i]; if (received_signals[i]) {
received_signals[i] = false;
struct sig_handler *handler = &fdm->signal_handlers[i];
xassert(handler->callback != NULL); xassert(handler->callback != NULL);
if (!handler->callback(fdm, i, handler->callback_data)) if (!handler->callback(fdm, i, handler->callback_data))
return false; return false;
}
} }
return true;
} }
}
LOG_ERRNO("failed to epoll"); if (unlikely(r < 0)) {
if (errno_copy == EINTR)
return true;
LOG_ERRNO_P(errno_copy, "failed to epoll");
return false; return false;
} }