mirrors/looking-glass
2
0
Fork 0
mirror of https://github.com/gnif/LookingGlass.git synced 2024-12-22 22:01:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

[common] open: detach xdg-open instead of waiting for it

Browse source 
Sometimes, e.g. when xdg-open has to start the browser, the xdg-open
process can stay around until the browser exits, which freezes the
client. Instead, we should not wait for xdg-open to exit.

However, we can't simply not call wait, as that would leave the
xdg-open process around as a zombie. We could turn off the SIGCHLD
handler, but that's a global solution to a local problem. Instead, we
call setsid and fork again to detach the xdg-open process as if it's a
daemon, and let init take care of the reaping process.

Co-Authored-By: Tudor Brindus <me@tbrindus.ca>
This commit is contained in:
Quantum 2021-09-23 23:07:15 -04:00 committed by Geoffrey McRae
parent bc65de5987
commit 57d220a43b
1 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
common/src/platform/linux/open.c
View file

@ -28,12 +28,19 @@
static bool xdgOpen(const char * path)
{
pid_t pid = fork();
if (pid == 0)
{
// setsid and fork again to detach the xdg-open process.
setsid();
pid_t pid = fork();
if (pid == 0)
{
execlp("xdg-open", "xdg-open", path, NULL);
_exit(127);
}
_exit(pid < 0);
}
else if (pid < 0)
{
DEBUG_ERROR("Failed to launch xdg-open: %s", strerror(errno));
@ -52,9 +59,9 @@ static bool xdgOpen(const char * path)
return true;
if (WIFEXITED(status))
DEBUG_ERROR("xdg-open exited with code %d", WEXITSTATUS(status));
DEBUG_ERROR("helper process exited with code %d", WEXITSTATUS(status));
else
DEBUG_ERROR("xdg-open exited with signal: %s", strsignal(WTERMSIG(status)));
DEBUG_ERROR("helper process exited with signal: %s", strsignal(WTERMSIG(status)));
return false;
}
}