Fixes segfault issue recorded in #406

tbranyen · tbranyen · commit c6c66a86fe93 · 2015-02-23T19:18:54.000-05:00
A segfault was occuring when passing null values to `NanNew`, this patch
will the bug for `const char *` and `unsigned int` C types.  We'll
definitely need to add more in the future.
diff --git a/generate/templates/partials/field_accessors.cc b/generate/templates/partials/field_accessors.cc
@@ -122,6 +122,20 @@
           return;
         }
 
+        {% each field.args|argsInfo as arg %}
+          {% if arg.name == "payload" %}
+            {%-- Do nothing --%}
+          {% elsif arg.isJsArg %}
+          if (baton->{{ arg.name }} == NULL) {
+            {% if arg.cType == "const char *" %}
+              baton->{{ arg.name }} = "";
+            {% elsif arg.cType == "unsigned int" %}
+              baton->{{ arg.name }} = 0;
+            {% endif %}
+          }
+          {% endif %}
+        {% endeach %}
+
         Local<Value> argv[{{ field.args|jsArgsCount }}] = {
           {% each field.args|argsInfo as arg %}
             {% if arg.name == "payload" %}
