Fix for issue 25236

runtime/vm/thread_registry.cc

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/thread_registry.h"
 
 #include "vm/isolate.h"
 #include "vm/lockers.h"
 
 namespace dart {
@@ skipping 58 matching lines @@
 Thread* ThreadRegistry::Schedule(Isolate* isolate,
                                  bool is_mutator,
                                  bool bypass_safepoint) {
   MonitorLocker ml(monitor_);
   // Wait for any rendezvous in progress.
   while (!bypass_safepoint && in_rendezvous_) {
     ml.Wait(Monitor::kNoTimeout);
   }
   Thread* thread = NULL;
   OSThread* os_thread = OSThread::Current();
-  ASSERT(os_thread != NULL);
-  ASSERT(isolate->heap() != NULL);
-  // First get a Thread structure. (we special case the mutator thread
-  // by reusing the cached structure, see comment in 'thread_registry.h').
-  if (is_mutator) {
-    if (mutator_thread_ == NULL) {
-      mutator_thread_ = GetThreadFromFreelist(isolate);
+  if (os_thread != NULL) {
+    ASSERT(isolate->heap() != NULL);
+    // First get a Thread structure. (we special case the mutator thread
+    // by reusing the cached structure, see comment in 'thread_registry.h').
+    if (is_mutator) {
+      if (mutator_thread_ == NULL) {
+        mutator_thread_ = GetThreadFromFreelist(isolate);
+      }
+      thread = mutator_thread_;
+    } else {
+      thread = GetThreadFromFreelist(isolate);
+      ASSERT(thread->api_top_scope() == NULL);
     }
-    thread = mutator_thread_;
-  } else {
-    thread = GetThreadFromFreelist(isolate);
-    ASSERT(thread->api_top_scope() == NULL);
+    // Now add this Thread to the active list for the isolate.
+    AddThreadToActiveList(thread);
+    // Set up other values and set the TLS value.
+    thread->isolate_ = isolate;
+    thread->heap_ = isolate->heap();
+    thread->set_os_thread(os_thread);
+    os_thread->set_thread(thread);
+    Thread::SetCurrent(thread);
+    os_thread->EnableThreadInterrupts();
   }
-  // Now add this Thread to the active list for the isolate.
-  AddThreadToActiveList(thread);
-  // Set up other values and set the TLS value.
-  thread->isolate_ = isolate;
-  thread->heap_ = isolate->heap();
-  thread->set_os_thread(os_thread);
-  os_thread->set_thread(thread);
-  Thread::SetCurrent(thread);
-  os_thread->EnableThreadInterrupts();
   return thread;
 }
 
 
 void ThreadRegistry::Unschedule(Thread* thread,
                                 bool is_mutator,
                                 bool bypass_safepoint) {
   MonitorLocker ml(monitor_);
   OSThread* os_thread = thread->os_thread();
   ASSERT(os_thread != NULL);
@@ skipping 126 matching lines @@
   intptr_t count = 0;
   Thread* current = active_list_;
   while (current != NULL) {
     ++count;
     current = current->next_;
   }
   return count;
 }
 
 }  // namespace dart