lib/libxml2
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/libxml2.git synced 2025-07-13 09:01:53 +03:00
Code Activity

adding a new API for Christian Glahn: xmlParseBalancedChunkMemoryRecover

Browse Source 
* parser.c include/libxml/parser.h: adding a new API for Christian
  Glahn: xmlParseBalancedChunkMemoryRecover
* valid.c: patch from Rick Jones for some grammar cleanup in
  validation messages
* result/VC/* result/valid/*: this slightly change some of the
  regression tests outputs
Daniel
This commit is contained in:
Daniel Veillard
2002-08-02 22:19:49 +00:00
parent 0bf2900b50
commit 58e44c9daf
14 changed files with 91 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
ChangeLog
View File

@ -1,3 +1,12 @@
Sat Aug 3 00:15:06 CEST 2002 Daniel Veillard <daniel@veillard.com>
* parser.c include/libxml/parser.h: adding a new API for Christian
Glahn: xmlParseBalancedChunkMemoryRecover
* valid.c: patch from Rick Jones for some grammar cleanup in
validation messages
* result/VC/* result/valid/*: this slightly change some of the
regression tests outputs
Thu Aug 1 14:50:28 CEST 2002 Daniel Veillard <daniel@veillard.com> Thu Aug 1 14:50:28 CEST 2002 Daniel Veillard <daniel@veillard.com>
* tree.c: trying to fix a problem in namespaced attribute handling * tree.c: trying to fix a problem in namespaced attribute handling

7
include/libxml/parser.h
View File

@ -767,6 +767,13 @@ int xmlParseBalancedChunkMemory(xmlDocPtr doc,
int depth, int depth,
const xmlChar *string, const xmlChar *string,
xmlNodePtr *lst); xmlNodePtr *lst);
int xmlParseBalancedChunkMemoryRecover(xmlDocPtr doc,
xmlSAXHandlerPtr sax,
void *user_data,
int depth,
const xmlChar *string,
xmlNodePtr *lst,
int recover);
int xmlParseExternalEntity (xmlDocPtr doc, int xmlParseExternalEntity (xmlDocPtr doc,
xmlSAXHandlerPtr sax, xmlSAXHandlerPtr sax,
void *user_data, void *user_data,

42
parser.c
View File

@ -9712,6 +9712,38 @@ xmlParseExternalEntity(xmlDocPtr doc, xmlSAXHandlerPtr sax, void *user_data,
int int
xmlParseBalancedChunkMemory(xmlDocPtr doc, xmlSAXHandlerPtr sax, xmlParseBalancedChunkMemory(xmlDocPtr doc, xmlSAXHandlerPtr sax,
void *user_data, int depth, const xmlChar *string, xmlNodePtr *lst) { void *user_data, int depth, const xmlChar *string, xmlNodePtr *lst) {
return xmlParseBalancedChunkMemoryRecover( doc, sax, user_data,
depth, string, lst, 0 );
}
/**
* xmlParseBalancedChunkMemoryRecover:
* @doc: the document the chunk pertains to
* @sax: the SAX handler bloc (possibly NULL)
* @user_data: The user data returned on SAX callbacks (possibly NULL)
* @depth: Used for loop detection, use 0
* @string: the input string in UTF8 or ISO-Latin (zero terminated)
* @lst: the return value for the set of parsed nodes
* @recover: return nodes even if the data is broken (use 0)
*
*
* Parse a well-balanced chunk of an XML document
* called by the parser
* The allowed sequence for the Well Balanced Chunk is the one defined by
* the content production in the XML grammar:
*
* [43] content ::= (element | CharData | Reference | CDSect | PI | Comment)*
*
* Returns 0 if the chunk is well balanced, -1 in case of args problem and
* the parser error code otherwise
*
* In case recover is set to 1, the nodelist will not be empty even if
* the parsed chunk is not well balanced.
*/
int
xmlParseBalancedChunkMemoryRecover(xmlDocPtr doc, xmlSAXHandlerPtr sax,
void *user_data, int depth, const xmlChar *string, xmlNodePtr *lst,
int recover) {
xmlParserCtxtPtr ctxt; xmlParserCtxtPtr ctxt;
xmlDocPtr newDoc; xmlDocPtr newDoc;
xmlSAXHandlerPtr oldsax = NULL; xmlSAXHandlerPtr oldsax = NULL;
@ -9806,8 +9838,11 @@ xmlParseBalancedChunkMemory(xmlDocPtr doc, xmlSAXHandlerPtr sax,
else else
ret = ctxt->errNo; ret = ctxt->errNo;
} else { } else {
if (lst != NULL) { ret = 0;
xmlNodePtr cur; }
if (lst != NULL && (ret == 0 || recover == 1)) {
xmlNodePtr cur;
/* /*
* Return the newly created nodeset after unlinking it from * Return the newly created nodeset after unlinking it from
@ -9821,8 +9856,7 @@ xmlParseBalancedChunkMemory(xmlDocPtr doc, xmlSAXHandlerPtr sax,
} }
newDoc->children->children = NULL; newDoc->children->children = NULL;
} }
ret = 0;
}
if (sax != NULL) if (sax != NULL)
ctxt->sax = oldsax; ctxt->sax = oldsax;
xmlFreeParserCtxt(ctxt); xmlFreeParserCtxt(ctxt);

8
result/VC/AttributeDefaultLegal
View File

@ -1,12 +1,12 @@
./test/VC/AttributeDefaultLegal:4: validity error: Attribute doc on At: invalid default value ./test/VC/AttributeDefaultLegal:4: validity error: Attribute doc of At: invalid default value
<!ATTLIST doc At NMTOKEN "$$$"> <!ATTLIST doc At NMTOKEN "$$$">
^ ^
./test/VC/AttributeDefaultLegal:6: validity error: Attribute doc on bad: invalid default value ./test/VC/AttributeDefaultLegal:6: validity error: Attribute doc of bad: invalid default value
<!ATTLIST doc bad IDREF "1abc_2"> <!ATTLIST doc bad IDREF "1abc_2">
^ ^
./test/VC/AttributeDefaultLegal:8: validity error: Attribute doc on bad2: invalid default value ./test/VC/AttributeDefaultLegal:8: validity error: Attribute doc of bad2: invalid default value
<!ATTLIST doc bad2 IDREFS "abc:1 1abc_2"> <!ATTLIST doc bad2 IDREFS "abc:1 1abc_2">
^ ^
./test/VC/AttributeDefaultLegal:11: validity error: No declaration for attribute val on element doc ./test/VC/AttributeDefaultLegal:11: validity error: No declaration for attribute val of element doc
<doc val="v1"/> <doc val="v1"/>
^ ^

2
result/VC/ElementValid5
View File

@ -1,4 +1,4 @@
./test/VC/ElementValid5:7: validity error: Element doc content doesn't follow the DTD ./test/VC/ElementValid5:7: validity error: Element doc content does not follow the DTD
Expecting (a , b* , c+), got (a b c b) Expecting (a , b* , c+), got (a b c b)
<doc><a/><b> but this</b><c>was not declared</c><b>seems</b></doc> <doc><a/><b> but this</b><c>was not declared</c><b>seems</b></doc>
^ ^

2
result/VC/ElementValid6
View File

@ -1,4 +1,4 @@
./test/VC/ElementValid6:7: validity error: Element doc content doesn't follow the DTD ./test/VC/ElementValid6:7: validity error: Element doc content does not follow the DTD
Expecting (a , b? , c+)?, got (a b) Expecting (a , b? , c+)?, got (a b)
<doc><a/><b>lacks c</b></doc> <doc><a/><b>lacks c</b></doc>
^ ^

2
result/VC/ElementValid7
View File

@ -1,4 +1,4 @@
./test/VC/ElementValid7:7: validity error: Element doc content doesn't follow the DTD ./test/VC/ElementValid7:7: validity error: Element doc content does not follow the DTD
Expecting ((a | b)* , c+ , a , b? , c , a?), got (a b a c c a) Expecting ((a | b)* , c+ , a , b? , c , a?), got (a b a c c a)
<doc><a/><b/><a/><c/><c/><a/></doc> <doc><a/><b/><a/><c/><c/><a/></doc>
^ ^

2
result/VC/ElementValid8
View File

@ -1,3 +1,3 @@
./test/VC/ElementValid8:7: validity warning: Element doc content model is ambiguous ./test/VC/ElementValid8:7: validity warning: Content model for Element doc is ambiguous
<doc><a/><c> doc is non-deterministic </c></doc> <doc><a/><c> doc is non-deterministic </c></doc>
^ ^

2
result/VC/Enumeration
View File

@ -1,3 +1,3 @@
./test/VC/Enumeration:5: validity error: Value "v4" for attribute val on doc is not among the enumerated set ./test/VC/Enumeration:5: validity error: Value "v4" for attribute val of doc is not among the enumerated set
<doc val="v4"></doc> <doc val="v4"></doc>
^ ^

2
result/VC/NS2
View File

@ -1,3 +1,3 @@
./test/VC/NS2:9: validity error: No declaration for attribute attr on element doc ./test/VC/NS2:9: validity error: No declaration for attribute attr of element doc
<ns:doc ns:attr="val" xmlns:ns="http://www.example.org/test/"> <ns:doc ns:attr="val" xmlns:ns="http://www.example.org/test/">
^ ^

2
result/valid/rss.xml.err
View File

@ -1,3 +1,3 @@
./test/valid/rss.xml:177: validity error: Element rss doesn't carry attribute version ./test/valid/rss.xml:177: validity error: Element rss does not carry attribute version
</rss> </rss>
^ ^

2
result/valid/xlink.xml.err
View File

@ -1,6 +1,6 @@
./test/valid/xlink.xml:450: validity error: ID dt-arc already defined ./test/valid/xlink.xml:450: validity error: ID dt-arc already defined
<p><termdef id="dt-arc" term="Arc">An <term>arc</term> is contained within an <p><termdef id="dt-arc" term="Arc">An <term>arc</term> is contained within an
^ ^
./test/valid/xlink.xml:530: validity error: IDREF attribute def reference an unknown ID "dt-xlg" ./test/valid/xlink.xml:530: validity error: IDREF attribute def references an unknown ID "dt-xlg"
^ ^

48
valid.c
View File

@ -1299,7 +1299,7 @@ xmlAddAttributeDecl(xmlValidCtxtPtr ctxt, xmlDtdPtr dtd, const xmlChar *elem,
} }
if ((defaultValue != NULL) && if ((defaultValue != NULL) &&
(!xmlValidateAttributeValue(type, defaultValue))) { (!xmlValidateAttributeValue(type, defaultValue))) {
VERROR(ctxt->userData, "Attribute %s on %s: invalid default value\n", VERROR(ctxt->userData, "Attribute %s of %s: invalid default value\n",
elem, name, defaultValue); elem, name, defaultValue);
defaultValue = NULL; defaultValue = NULL;
ctxt->valid = 0; ctxt->valid = 0;
@ -1362,7 +1362,7 @@ xmlAddAttributeDecl(xmlValidCtxtPtr ctxt, xmlDtdPtr dtd, const xmlChar *elem,
* The attribute is already defined in this DTD. * The attribute is already defined in this DTD.
*/ */
VWARNING(ctxt->userData, VWARNING(ctxt->userData,
"Attribute %s on %s: already defined\n", "Attribute %s of element %s: already defined\n",
name, elem); name, elem);
xmlFreeAttribute(ret); xmlFreeAttribute(ret);
return(NULL); return(NULL);
@ -3203,7 +3203,7 @@ xmlValidateAttributeDecl(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
val = xmlValidateAttributeValue(attr->atype, attr->defaultValue); val = xmlValidateAttributeValue(attr->atype, attr->defaultValue);
if (val == 0) { if (val == 0) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Syntax of default value for attribute %s on %s is not valid\n", "Syntax of default value for attribute %s of %s is not valid\n",
attr->name, attr->elem); attr->name, attr->elem);
} }
ret &= val; ret &= val;
@ -3214,7 +3214,7 @@ xmlValidateAttributeDecl(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
(attr->def != XML_ATTRIBUTE_IMPLIED) && (attr->def != XML_ATTRIBUTE_IMPLIED) &&
(attr->def != XML_ATTRIBUTE_REQUIRED)) { (attr->def != XML_ATTRIBUTE_REQUIRED)) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"ID attribute %s on %s is not valid must be #IMPLIED or #REQUIRED\n", "ID attribute %s of %s is not valid must be #IMPLIED or #REQUIRED\n",
attr->name, attr->elem); attr->name, attr->elem);
ret = 0; ret = 0;
} }
@ -3271,7 +3271,7 @@ xmlValidateAttributeDecl(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
} }
if (tree == NULL) { if (tree == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Default value \"%s\" for attribute %s on %s is not among the enumerated set\n", "Default value \"%s\" for attribute %s of %s is not among the enumerated set\n",
attr->defaultValue, attr->name, attr->elem); attr->defaultValue, attr->name, attr->elem);
ret = 0; ret = 0;
} }
@ -3445,7 +3445,7 @@ xmlValidateOneAttribute(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
/* Validity Constraint: Attribute Value Type */ /* Validity Constraint: Attribute Value Type */
if (attrDecl == NULL) { if (attrDecl == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"No declaration for attribute %s on element %s\n", "No declaration for attribute %s of element %s\n",
attr->name, elem->name); attr->name, elem->name);
return(0); return(0);
} }
@ -3454,7 +3454,7 @@ xmlValidateOneAttribute(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
val = xmlValidateAttributeValue(attrDecl->atype, value); val = xmlValidateAttributeValue(attrDecl->atype, value);
if (val == 0) { if (val == 0) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Syntax of value for attribute %s on %s is not valid\n", "Syntax of value for attribute %s of %s is not valid\n",
attr->name, elem->name); attr->name, elem->name);
ret = 0; ret = 0;
} }
@ -3463,7 +3463,7 @@ xmlValidateOneAttribute(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
if (attrDecl->def == XML_ATTRIBUTE_FIXED) { if (attrDecl->def == XML_ATTRIBUTE_FIXED) {
if (!xmlStrEqual(value, attrDecl->defaultValue)) { if (!xmlStrEqual(value, attrDecl->defaultValue)) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Value for attribute %s on %s is different from default \"%s\"\n", "Value for attribute %s of %s is different from default \"%s\"\n",
attr->name, elem->name, attrDecl->defaultValue); attr->name, elem->name, attrDecl->defaultValue);
ret = 0; ret = 0;
} }
@ -3493,7 +3493,7 @@ xmlValidateOneAttribute(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
if (nota == NULL) { if (nota == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Value \"%s\" for attribute %s on %s is not a declared Notation\n", "Value \"%s\" for attribute %s of %s is not a declared Notation\n",
value, attr->name, elem->name); value, attr->name, elem->name);
ret = 0; ret = 0;
} }
@ -3505,7 +3505,7 @@ xmlValidateOneAttribute(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
} }
if (tree == NULL) { if (tree == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Value \"%s\" for attribute %s on %s is not among the enumerated notations\n", "Value \"%s\" for attribute %s of %s is not among the enumerated notations\n",
value, attr->name, elem->name); value, attr->name, elem->name);
ret = 0; ret = 0;
} }
@ -3520,7 +3520,7 @@ xmlValidateOneAttribute(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
} }
if (tree == NULL) { if (tree == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Value \"%s\" for attribute %s on %s is not among the enumerated set\n", "Value \"%s\" for attribute %s of %s is not among the enumerated set\n",
value, attr->name, elem->name); value, attr->name, elem->name);
ret = 0; ret = 0;
} }
@ -3530,7 +3530,7 @@ xmlValidateOneAttribute(xmlValidCtxtPtr ctxt, xmlDocPtr doc,
if ((attrDecl->def == XML_ATTRIBUTE_FIXED) && if ((attrDecl->def == XML_ATTRIBUTE_FIXED) &&
(!xmlStrEqual(attrDecl->defaultValue, value))) { (!xmlStrEqual(attrDecl->defaultValue, value))) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Value for attribute %s on %s must be \"%s\"\n", "Value for attribute %s of %s must be \"%s\"\n",
attr->name, elem->name, attrDecl->defaultValue); attr->name, elem->name, attrDecl->defaultValue);
ret = 0; ret = 0;
} }
@ -4082,7 +4082,7 @@ xmlValidateElementContent(xmlValidCtxtPtr ctxt, xmlNodePtr child,
ret = xmlValidateElementType(ctxt); ret = xmlValidateElementType(ctxt);
if ((ret == -3) && (warn)) { if ((ret == -3) && (warn)) {
VWARNING(ctxt->userData, VWARNING(ctxt->userData,
"Element %s content model is ambiguous\n", name); "Content model for Element %s is ambiguous\n", name);
} else if (ret == -2) { } else if (ret == -2) {
/* /*
* An entities reference appeared at this level. * An entities reference appeared at this level.
@ -4185,21 +4185,21 @@ xmlValidateElementContent(xmlValidCtxtPtr ctxt, xmlNodePtr child,
if (name != NULL) { if (name != NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Element %s content doesn't follow the DTD\nExpecting %s, got %s\n", "Element %s content does not follow the DTD\nExpecting %s, got %s\n",
name, expr, list); name, expr, list);
} else { } else {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Element content doesn't follow the DTD\nExpecting %s, got %s\n", "Element content does not follow the DTD\nExpecting %s, got %s\n",
expr, list); expr, list);
} }
} else { } else {
if (name != NULL) { if (name != NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Element %s content doesn't follow the DTD\n", "Element %s content does not follow the DTD\n",
name); name);
} else { } else {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Element content doesn't follow the DTD\n"); "Element content does not follow the DTD\n");
} }
} }
ret = 0; ret = 0;
@ -4620,12 +4620,12 @@ child_ok:
if (qualified == -1) { if (qualified == -1) {
if (attr->prefix == NULL) { if (attr->prefix == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Element %s doesn't carry attribute %s\n", "Element %s does not carry attribute %s\n",
elem->name, attr->name); elem->name, attr->name);
ret = 0; ret = 0;
} else { } else {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Element %s doesn't carry attribute %s:%s\n", "Element %s does not carry attribute %s:%s\n",
elem->name, attr->prefix,attr->name); elem->name, attr->prefix,attr->name);
ret = 0; ret = 0;
} }
@ -4669,7 +4669,7 @@ child_ok:
if (xmlStrEqual(attr->name, ns->prefix)) { if (xmlStrEqual(attr->name, ns->prefix)) {
if (!xmlStrEqual(attr->defaultValue, ns->href)) { if (!xmlStrEqual(attr->defaultValue, ns->href)) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Element %s namespace name for %s doesn't match the DTD\n", "Element %s namespace name for %s does not match the DTD\n",
elem->name, ns->prefix); elem->name, ns->prefix);
ret = 0; ret = 0;
} }
@ -4732,7 +4732,7 @@ xmlValidateRoot(xmlValidCtxtPtr ctxt, xmlDocPtr doc) {
(xmlStrEqual(root->name, BAD_CAST "html"))) (xmlStrEqual(root->name, BAD_CAST "html")))
goto name_ok; goto name_ok;
VERROR(ctxt->userData, VERROR(ctxt->userData,
"Not valid: root and DtD name do not match '%s' and '%s'\n", "Not valid: root and DTD name do not match '%s' and '%s'\n",
root->name, doc->intSubset->name); root->name, doc->intSubset->name);
return(0); return(0);
@ -4820,7 +4820,7 @@ xmlValidateRef(xmlRefPtr ref, xmlValidCtxtPtr ctxt,
id = xmlGetID(ctxt->doc, name); id = xmlGetID(ctxt->doc, name);
if (id == NULL) { if (id == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"IDREF attribute %s reference an unknown ID \"%s\"\n", "IDREF attribute %s references an unknown ID \"%s\"\n",
attr->name, name); attr->name, name);
ctxt->valid = 0; ctxt->valid = 0;
} }
@ -4841,7 +4841,7 @@ xmlValidateRef(xmlRefPtr ref, xmlValidCtxtPtr ctxt,
id = xmlGetID(ctxt->doc, str); id = xmlGetID(ctxt->doc, str);
if (id == NULL) { if (id == NULL) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"IDREFS attribute %s reference an unknown ID \"%s\"\n", "IDREFS attribute %s references an unknown ID \"%s\"\n",
attr->name, str); attr->name, str);
ctxt->valid = 0; ctxt->valid = 0;
} }
@ -5050,7 +5050,7 @@ xmlValidateAttributeCallback(xmlAttributePtr cur, xmlValidCtxtPtr ctxt,
} }
if (elem->etype == XML_ELEMENT_TYPE_EMPTY) { if (elem->etype == XML_ELEMENT_TYPE_EMPTY) {
VERROR(ctxt->userData, VERROR(ctxt->userData,
"NOTATION attribute %s declared on EMPTY element %s\n", "NOTATION attribute %s declared for EMPTY element %s\n",
cur->name, cur->elem); cur->name, cur->elem);
ctxt->valid = 0; ctxt->valid = 0;
} }

1
xinclude.c
View File

@ -32,6 +32,7 @@
#define XINCLUDE_NS (const xmlChar *) "http://www.w3.org/2001/XInclude" #define XINCLUDE_NS (const xmlChar *) "http://www.w3.org/2001/XInclude"
#define XINCLUDE_NODE (const xmlChar *) "include" #define XINCLUDE_NODE (const xmlChar *) "include"
#define XINCLUDE_FALLBACK (const xmlChar *) "fallback"
#define XINCLUDE_HREF (const xmlChar *) "href" #define XINCLUDE_HREF (const xmlChar *) "href"
#define XINCLUDE_PARSE (const xmlChar *) "parse" #define XINCLUDE_PARSE (const xmlChar *) "parse"
#define XINCLUDE_PARSE_XML (const xmlChar *) "xml" #define XINCLUDE_PARSE_XML (const xmlChar *) "xml"