wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1163135 - Make it safer to start the profiler via signal; r=snorp

Browse Source
This commit is contained in:
Jim Chen 2015-05-19 11:27:18 -04:00
parent 0aefa6e182
commit 1807082f8f
1 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
tools/profiler/platform-linux.cc
View File

@ -646,11 +646,7 @@ static void ReadProfilerVars(const char* fileName, const char** features,
}
}
static void StartSignalHandler(int signal, siginfo_t* info, void* context) {
// XXX: Everything we do here is NOT async signal safe. We risk nasty things
// like deadlocks but we typically only do this once so it tends to be ok.
// See bug 909403
static void DoStartTask() {
uint32_t featureCount = 0;
uint32_t threadCount = 0;
@ -674,6 +670,20 @@ static void StartSignalHandler(int signal, siginfo_t* info, void* context) {
freeArray(features, featureCount);
}
static void StartSignalHandler(int signal, siginfo_t* info, void* context) {
class StartTask : public nsRunnable {
public:
NS_IMETHOD Run() {
DoStartTask();
return NS_OK;
}
};
// XXX: technically NS_DispatchToMainThread is NOT async signal safe. We risk
// nasty things like deadlocks, but the probability is very low and we
// typically only do this once so it tends to be ok. See bug 909403.
NS_DispatchToMainThread(new StartTask());
}
void OS::Startup()
{
LOG("Registering start signal");