Passing NULL to wrenInterpret for Module will crash.

[IngwiePhoenix]

I know, you shouldn't do that - but out of curiosity, I tried it... and, as expected, it wouldn't work, but instead crash the entire application. Here is what I did.

Ingwie@Ingwies-Macbook-Pro.local ~/W/G/w/out $ cat test.c
#include <stdio.h>
#include "wren.h"

void printOut(WrenVM* vm, const char* text) {
  printf("%s", text);
}

void printErr(
    WrenVM* vm, WrenErrorType type,
    const char* module, int line,
    const char* message
) {
  switch(type) {
    case WREN_ERROR_RUNTIME:
      printf("[wren] ERR: Runtime reported: \"%s\"\n", message);
      return;
    case WREN_ERROR_COMPILE:
      printf("[wren] ERR: (%s:%i) %s\n", module, line, message);
      return;
    case WREN_ERROR_STACK_TRACE:
      printf("[wren] STACK: %s:%i = %s", module, line, message);
      return;
  }
}

const char* rootCode =
"class A {\n"
"  construct new() { System.print('Constructed'); }\n"
"}\n";

const char* script =
"var a = A.new()";

int main(int argc, const char** argv) {
  WrenVM* vm;
  WrenConfiguration config;
  wrenInitConfiguration(&config);

  config.writeFn = printOut;
  config.errorFn = printErr;

  vm = wrenNewVM(&config);

  WrenInterpretResult rt;

  printf("Root: \n");
  rt = wrenInterpret(vm, NULL, rootCode);
  if(rt != WREN_RESULT_SUCCESS) exit(1);

  printf("Main: \n");
  rt = wrenInterpret(vm, "internal:main", script);
  if(rt != WREN_RESULT_SUCCESS) exit(1);

  wrenFreeVM(vm);
  exit(0);
}
Ingwie@Ingwies-Macbook-Pro.local ~/W/G/w/out $ gcc -I ../src/include/ ../lib/libwren.a test.c -o test
Ingwie@Ingwies-Macbook-Pro.local ~/W/G/w/out $ ./test
Root: 
fish: './test' terminated by signal SIGSEGV (Address boundary error)

Might want a sanity check if module is really not NULL?