ext: use xmlFree() instead of free() in Document#set_encoding

to match the allocation done by xmlStrdup() Related to e221b2c
sparklemotion · Aug 6, 2021 · 1ef7dff · 1ef7dff
1 parent b016ff3
commit 1ef7dff
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/ext/nokogiri/xml_document.c b/ext/nokogiri/xml_document.c
@@ -213,7 +213,7 @@ set_encoding(VALUE self, VALUE encoding)
   Data_Get_Struct(self, xmlDoc, doc);
 
   if (doc->encoding) {
-    free((char *)(uintptr_t) doc->encoding);  /* avoid gcc cast warning */
+    xmlFree((xmlChar *)(uintptr_t) doc->encoding);  /* avoid gcc cast warning */
   }
 
   doc->encoding = xmlStrdup((xmlChar *)StringValueCStr(encoding));