commit dda22c15d5447b4a2a7bd47d456bcc5bbcef2472
author Daniel Veillard <veillard@src.gnome.org> Sat Jan 24 08:31:30 2004 +0000
committer Daniel Veillard <veillard@src.gnome.org> Sat Jan 24 08:31:30 2004 +0000
tree 8c759e859af68a96b801228741066689c1e9ddff
parent 4773df2a58be83e1b9b1b55840371acf37386820

applied patch from Steve Ball to avoid a double-free. Daniel

* include/libxml/schemasInternals.h xmlschemas.c: applied patch from
  Steve Ball to avoid a double-free.
Daniel

xmlschemas.c

diff --git a/xmlschemas.c b/xmlschemas.c
index d639b67..f9e5153 100644
--- a/xmlschemas.c
+++ b/xmlschemas.c
@@ -85,6 +85,7 @@
 
     const xmlChar *URL;
     xmlDocPtr doc;
+    int preserve;		/* Whether the doc should be freed  */
 
     const char *buffer;
     int size;
@@ -650,7 +651,7 @@
     }
     if (schema->annot != NULL)
         xmlSchemaFreeAnnot(schema->annot);
-    if (schema->doc != NULL)
+    if (schema->doc != NULL && !schema->preserve)
         xmlFreeDoc(schema->doc);
     xmlDictFree(schema->dict);
 
@@ -3840,6 +3841,8 @@
     memset(ret, 0, sizeof(xmlSchemaParserCtxt));
     ret->doc = doc;
     ret->dict = xmlDictCreate();
+    /* The application has responsibility for the document */
+    ret->preserve = 1;
 
     return (ret);
 }
@@ -3855,7 +3858,7 @@
 {
     if (ctxt == NULL)
         return;
-    if (ctxt->doc != NULL)
+    if (ctxt->doc != NULL && !ctxt->preserve)
         xmlFreeDoc(ctxt->doc);
     xmlDictFree(ctxt->dict);
     xmlFree(ctxt);
@@ -4862,6 +4865,7 @@
     xmlDocPtr doc;
     xmlNodePtr root;
     int nberrors;
+    int preserve = 0;
 
     xmlSchemaInitTypes();
 
@@ -4900,6 +4904,7 @@
         ctxt->URL = xmlDictLookup(ctxt->dict, BAD_CAST "in_memory_buffer", -1);
     } else if (ctxt->doc != NULL) {
         doc = ctxt->doc;
+	preserve = 1;
     } else {
 	xmlSchemaPErr(ctxt, NULL,
 		      XML_SCHEMAP_NOTHING_TO_PARSE,
@@ -4916,7 +4921,9 @@
 	xmlSchemaPErr(ctxt, (xmlNodePtr) doc,
 		      XML_SCHEMAP_NOROOT,
 		      "schemas has no root", NULL, NULL);
-	xmlFreeDoc(doc);
+	if (!preserve) {
+	    xmlFreeDoc(doc);
+	}
         return (NULL);
     }
 
@@ -4930,10 +4937,13 @@
      */
     ret = xmlSchemaParseSchema(ctxt, root);
     if (ret == NULL) {
-	xmlFreeDoc(doc);
+        if (!preserve) {
+	    xmlFreeDoc(doc);
+	}
         return (NULL);
     }
     ret->doc = doc;
+    ret->preserve = preserve;
 
     /*
      * Then fix all the references.