1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QProcess/Unix: allow startDetached() to also use vfork()

Browse Source 
The same arguments that applied to the regular process-starting code in
commit e1a787a76ed462e4ed49db78a40c6d7e272182d7 (6.4) apply here too.

Task-number: QTBUG-104493
Change-Id: Icfe44ecf285a480fafe4fffd174da1b05d0054bc
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@gmx.de>
Reviewed-by: Marcus Tillmanns <marcus.tillmanns@qt.io>
This commit is contained in:
Thiago Macieira 2023-03-18 14:46:54 -07:00
parent 815bb2a4fb
commit c5c7712910
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/corelib/io/qprocess_unix.cpp
View File

@ -1090,7 +1090,12 @@ bool QProcessPrivate::startDetached(qint64 *pid)
// see startProcess() for more information // see startProcess() for more information
PThreadCancelGuard cancelGuard; PThreadCancelGuard cancelGuard;
pid_t childPid = fork(); auto doFork = [this]() {
if (useForkFlags(unixExtras.get()))
return fork;
QT_IGNORE_DEPRECATIONS(return vfork;)
}();
pid_t childPid = doFork();
if (childPid == 0) { if (childPid == 0) {
::signal(SIGPIPE, SIG_DFL); // reset the signal that we ignored ::signal(SIGPIPE, SIG_DFL); // reset the signal that we ignored
::setsid(); ::setsid();
@ -1105,7 +1110,7 @@ bool QProcessPrivate::startDetached(qint64 *pid)
::_exit(1); ::_exit(1);
}; };
pid_t doubleForkPid = fork(); pid_t doubleForkPid = doFork();
if (doubleForkPid == 0) { if (doubleForkPid == 0) {
// Render channels configuration. // Render channels configuration.
commitChannels(); commitChannels();