Avoid leaking orphaned VM instances.

When starting a Thread, there is a point where resources are created but an exception could be raised that would result in the resources not getting reclaimed. At the moment, this is the VM instance. This will be refactored soon, but the same caution will always apply to creating Thread instances.
rubinius · Mar 27, 2016 · 045796e · 045796e
1 parent 4c0c5b4
commit 045796e
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/machine/builtin/thread.cpp b/machine/builtin/thread.cpp
@@ -156,6 +156,7 @@ namespace rubinius {
         call_frame->line(state));
 
     if(!thread->send(state, state->symbol("initialize"), args, block, true)) {
+      thread->vm()->set_zombie(state);
       return NULL;
     }
 
@@ -175,6 +176,7 @@ namespace rubinius {
         call_frame->line(state));
 
     if(!thread->send(state, state->symbol("__thread_initialize__"), args, block, true)) {
+      thread->vm()->set_zombie(state);
       return NULL;
     }