Bug 927979 - Follow up to fix bad string null check in profiler. r=benwa
authorJim Mathies <jmathies@mozilla.com>
Mon, 04 Nov 2013 12:27:40 -0600
changeset 153417 1d65a43d4ed0cb4c6245b64e633318a718273a97
parent 153416 92f44bb34cc858911b26538dd0cadee6875d3e1c
child 153418 30ad8d9013616538f20673600dd284f7b85b8c68
push id25587
push userkwierso@gmail.com
push dateTue, 05 Nov 2013 05:07:06 +0000
treeherdermozilla-central@b0bce439c6e2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbenwa
bugs927979
milestone28.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 927979 - Follow up to fix bad string null check in profiler. r=benwa
tools/profiler/platform.cpp
--- a/tools/profiler/platform.cpp
+++ b/tools/profiler/platform.cpp
@@ -422,17 +422,17 @@ void set_tls_stack_top(void* stackTop)
   uintptr_t stackTopR = (uintptr_t)stackTop;
   if (stackTop) {
     stackTopR = (stackTopR & ~(uintptr_t)4095) + (uintptr_t)4095;
   }
   tlsStackTop.set((void*)stackTopR);
 }
 
 bool is_main_thread_name(const char* aName) {
-  if (aName) {
+  if (!aName) {
     return false;
   }
   return strcmp(aName, gGeckoThreadName) == 0;
 }
 
 ////////////////////////////////////////////////////////////////////////
 // BEGIN externally visible functions