Revision: 12050
Author:   [email protected]
Date:     Thu Jul 12 00:59:41 2012
Log:      Fix bug in CompilationHandlescope constructor.

This will never be triggered in the state v8 is currently in, but will crash v8 when multiple DeferredHandles are queued up during parallel compilation.

BUG=
TEST=

Review URL: https://chromiumcodereview.appspot.com/10696178
http://code.google.com/p/v8/source/detail?r=12050

Modified:
 /branches/bleeding_edge/src/api.h

=======================================
--- /branches/bleeding_edge/src/api.h   Fri Jul  6 07:09:11 2012
+++ /branches/bleeding_edge/src/api.h   Thu Jul 12 00:59:41 2012
@@ -399,8 +399,12 @@
  private:
   DeferredHandles(DeferredHandles* next, Object** first_block_limit,
                   HandleScopeImplementer* impl)
- : next_(next), previous_(NULL), first_block_limit_(first_block_limit),
-        impl_(impl) {}
+      : next_(next),
+        previous_(NULL),
+        first_block_limit_(first_block_limit),
+        impl_(impl) {
+    if (next != NULL) next->previous_ = this;
+  }

   void Iterate(ObjectVisitor* v);

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to