Don't crash on no error returned.

cfis · cfis · commit d8324b0ecf10 · 2024-01-07T21:36:18.000-08:00
diff --git a/ext/libxml/ruby_xml_error.c b/ext/libxml/ruby_xml_error.c
@@ -160,9 +160,12 @@ static VALUE rxml_error_reset_handler(VALUE self)
 
 NORETURN(void rxml_raise(const xmlError *xerror))
 {
-  /* Wrap error up as Ruby object and send it off to ruby */
-  VALUE error = rxml_error_wrap(xerror);
-  rb_exc_raise(error);
+  if (xerror)
+  {
+    /* Wrap error up as Ruby object and send it off to ruby */
+    VALUE error = rxml_error_wrap(xerror);
+    rb_exc_raise(error);
+  }
 }
 
 void rxml_init_error(void)

Original file line number	Diff line number	Diff line change
`@@ -160,9 +160,12 @@ static VALUE rxml_error_reset_handler(VALUE self)`
`160`	`160`
`161`	`161`	`NORETURN(void rxml_raise(const xmlError *xerror))`
`162`	`162`	`{`
`163`		`- /* Wrap error up as Ruby object and send it off to ruby */`
`164`		`- VALUE error = rxml_error_wrap(xerror);`
`165`		`- rb_exc_raise(error);`
	`163`	`+ if (xerror)`
	`164`	`+ {`
	`165`	`+ /* Wrap error up as Ruby object and send it off to ruby */`
	`166`	`+ VALUE error = rxml_error_wrap(xerror);`
	`167`	`+ rb_exc_raise(error);`
	`168`	`+ }`
`166`	`169`	`}`
`167`	`170`
`168`	`171`	`void rxml_init_error(void)`