Don’t initialize arguments on frame creation, only the stack and locals

Though, need to make sure I initialize the bootstrap frame arguments. Otherwise, we may run into uninitialized memory.

Signed-off-by: Stefan Marr <[email protected]>

src/vm/Universe.cpp

@@ -302,6 +302,11 @@ vm_oop_t Universe::interpretMethod(VMObject* receiver, VMInvokable* initialize,
     VMMethod* bootstrapMethod = createBootstrapMethod(load_ptr(systemClass), 2);
 
     VMFrame* bootstrapFrame = Interpreter::PushNewFrame(bootstrapMethod);
+    for (size_t argIdx = 0; argIdx < bootstrapMethod->GetNumberOfArguments();
+         argIdx += 1) {
+        bootstrapFrame->SetArgument((long)argIdx, (long)0, load_ptr(nilObject));
+    }
+
     bootstrapFrame->Push(receiver);
 
     if (argumentsArray != nullptr) {
@@ -720,7 +725,8 @@ VMFrame* Universe::NewFrame(VMFrame* previousFrame, VMMethod* method) {
                     method->GetMaximumNumberOfStackElements();
 
     size_t additionalBytes = length * sizeof(VMObject*);
-    result = new (GetHeap<HEAP_CLS>(), additionalBytes) VMFrame(additionalBytes, method, previousFrame);
+    result = new (GetHeap<HEAP_CLS>(), additionalBytes)
+        VMFrame(additionalBytes, method, previousFrame);
 
     LOG_ALLOCATION("VMFrame", result->GetObjectSize());
     return result;

src/vmobjects/VMFrame.h

@@ -54,8 +54,8 @@ class VMFrame : public VMObject {
         // --> until end of Frame
         gc_oop_t* end = (gc_oop_t*)SHIFTED_PTR(this, totalObjectSize);
         size_t i = 0;
-        while (arguments + i < end) {
-            arguments[i] = nilObject;
+        while (locals + i < end) {
+            locals[i] = nilObject;
             i++;
         }
     }