Avoid accessing T_NONE object in rxml_node_deregisterNode

Doing so causes segfault. sparklemotion/nokogiri#881 (comment)
increments · Sep 9, 2015 · 4141b4d · 4141b4d
1 parent d4a7298
commit 4141b4d
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/ext/libxml/ruby_xml_node.c b/ext/libxml/ruby_xml_node.c
@@ -49,6 +49,11 @@ static void rxml_node_deregisterNode(xmlNodePtr xnode)
       then disable the dfree function so that Ruby will not
       try to free the node a second time. */
     VALUE node = (VALUE) xnode->_private;
+
+    if (TYPE(node) == T_NONE) {
+        return;
+    }
+
     RDATA(node)->data = NULL;
     RDATA(node)->dfree = NULL;
     RDATA(node)->dmark = NULL;