- moved the hook-recursion-protection to tls

- some code cleanup and consolidation
- hook.dll will now report all of its own exceptions
- some more logging during startup
- changed the way urls are encoded for download requests
- now displaying (one of the) process name(s) while waiting for a program to end
- bugfix: spawned processes were forced to leave the job

1 files changed, 13 insertions, 0 deletions

diff --git a/src/spawn.cpp b/src/spawn.cpp
index 1392ec88..64f4e995 100644
--- a/src/spawn.cpp
+++ b/src/spawn.cpp
@@ -117,6 +117,19 @@ HANDLE startBinary(const QFileInfo &binary,
                    HANDLE stdOut,

                    HANDLE stdErr)

 {

+  JOBOBJECT_EXTENDED_LIMIT_INFORMATION jobInfo;

+

+  ::QueryInformationJobObject(NULL, JobObjectExtendedLimitInformation, &jobInfo, sizeof(JOBOBJECT_EXTENDED_LIMIT_INFORMATION), NULL);

+  jobInfo.BasicLimitInformation.LimitFlags |= JOB_OBJECT_LIMIT_BREAKAWAY_OK;

+

+  HANDLE jobObject = ::CreateJobObject(NULL, NULL);

+

+  if (jobObject == NULL) {

+    qWarning("failed to create job object: %lu", ::GetLastError());

+  } else {

+    ::SetInformationJobObject(jobObject, JobObjectExtendedLimitInformation, &jobInfo, sizeof(JOBOBJECT_EXTENDED_LIMIT_INFORMATION));

+  }

+

   HANDLE processHandle, threadHandle;

   std::wstring binaryName = ToWString(QDir::toNativeSeparators(binary.absoluteFilePath()));

   std::wstring currentDirectoryName = ToWString(QDir::toNativeSeparators(currentDirectory.absolutePath()));