Don't access null pointers when printing null module names

ruby0x1 · ruby0x1 · commit 688691b17c4d · 2019-09-18T22:49:01.000-07:00
fixes wren-lang#631
diff --git a/src/vm/wren_compiler.c b/src/vm/wren_compiler.c
@@ -393,18 +393,21 @@ static void printError(Parser* parser, int line, const char* label,
 {
   parser->hasError = true;
   if (!parser->printErrors) return;
-  
+
   // Only report errors if there is a WrenErrorFn to handle them.
   if (parser->vm->config.errorFn == NULL) return;
-  
+
   // Format the label and message.
   char message[ERROR_MESSAGE_SIZE];
   int length = sprintf(message, "%s: ", label);
   length += vsprintf(message + length, format, args);
   ASSERT(length < ERROR_MESSAGE_SIZE, "Error should not exceed buffer.");
-  
+
+  ObjString* module = parser->module->name;
+  const char* module_name = module ? module->value : "<unknown>";
+
   parser->vm->config.errorFn(parser->vm, WREN_ERROR_COMPILE,
-                             parser->module->name->value, line, message);
+                             module_name, line, message);
 }
 
 // Outputs a lexical error.
