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

term: remove client application’s pid from reaper when shutting down

Browse source 
When we are shutting down the terminal, we explicitly wait for the
child application to terminate (with a timeout, after which the child
process is killed).

I.e. there’s no need to let the reaper handle it. In fact, doing so
leads to a crash since we will have destroyed (and thus free:d) the
terminal instance when the reaper callback is called.
This commit is contained in:
Daniel Eklöf 2021-01-12 09:21:01 +01:00
parent 1d6b7d7a35
commit e563c863d1
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F
1 changed files with 3 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

3
terminal.c
View file

@ -1305,6 +1305,9 @@ term_shutdown(struct terminal *term)
fdm_del(term->fdm, term->blink.fd); fdm_del(term->fdm, term->blink.fd);
fdm_del(term->fdm, term->flash.fd); fdm_del(term->fdm, term->flash.fd);
/* We’ll deal with this explicitly */
reaper_del(term->reaper, term->slave);
if (term->window != NULL && term->window->is_configured) if (term->window != NULL && term->window->is_configured)
fdm_del(term->fdm, term->ptmx); fdm_del(term->fdm, term->ptmx);
else else