guard the type of define_function's first arg

Author: hmsk

ext/quickjsrb/quickjsrb.c

@@ -628,6 +628,11 @@ static VALUE vm_m_defineGlobalFunction(int argc, VALUE *argv, VALUE r_self)
   VALUE r_block;
   rb_scan_args(argc, argv, "10*&", &r_name, &r_flags, &r_block);
 
+  if (!(SYMBOL_P(r_name) || RB_TYPE_P(r_name, T_STRING)))
+  {
+    rb_raise(rb_eTypeError, "function's name should be a Symbol or a String");
+  }
+
   VMData *data;
   TypedData_Get_Struct(r_self, VMData, &vm_type, data);
 

test/quickjs_test.rb

@@ -335,6 +335,12 @@ class GlobalFunction < QuickjsVmTest
         assert(@vm.eval_code('sym()'))
       end
 
+      test "function's name can't be others than a symbol nor a string" do
+        assert_raise_with_message(TypeError, "function's name should be a Symbol or a String") do
+          @vm.define_function([:sym_in_ary]) { 'never reach' }
+        end
+      end
+
       [
         ["'symsym'", :symsym],
         ["null", nil],