Clone VMHashInstance.storage before building an Iterator.

Otherwise we get ConcurrentModificationException thrown when changing a Hash while iterating over it.
Raku · Sep 16, 2015 · 4740820 · 4740820
1 parent 0c380a8
commit 4740820
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java b/src/vm/jvm/runtime/org/perl6/nqp/runtime/Ops.java
@@ -3391,7 +3391,7 @@ else if (agg.st.REPR instanceof VMHash) {
             SixModelObject iterType = tc.curFrame.codeRef.staticInfo.compUnit.hllConfig.hashIteratorType;
             VMIterInstance iter = (VMIterInstance)iterType.st.REPR.allocate(tc, iterType.st);
             iter.target = agg;
-            iter.hashKeyIter = ((VMHashInstance)agg).storage.keySet().iterator();
+            iter.hashKeyIter = ((HashMap)((VMHashInstance)agg).storage.clone()).keySet().iterator();
             iter.iterMode = VMIterInstance.MODE_HASH;
             return iter;
         }