You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xerces.apache.org by tn...@apache.org on 2003/01/16 22:30:14 UTC
cvs commit: xml-xerces/c/src/xercesc/validators/DTD DTDScanner.cpp
tng 2003/01/16 13:30:14
Modified: c/src/xercesc/validators/DTD DTDScanner.cpp
Log:
[Bug 16151] Memory leak in DTDScanner with ill-formed DTD declaration. Fix by David Bertoni.
Revision Changes Path
1.21 +6 -3 xml-xerces/c/src/xercesc/validators/DTD/DTDScanner.cpp
Index: DTDScanner.cpp
===================================================================
RCS file: /home/cvs/xml-xerces/c/src/xercesc/validators/DTD/DTDScanner.cpp,v
retrieving revision 1.20
retrieving revision 1.21
diff -u -r1.20 -r1.21
--- DTDScanner.cpp 24 Dec 2002 16:12:19 -0000 1.20
+++ DTDScanner.cpp 16 Jan 2003 21:30:14 -0000 1.21
@@ -56,6 +56,9 @@
/*
* $Log$
+ * Revision 1.21 2003/01/16 21:30:14 tng
+ * [Bug 16151] Memory leak in DTDScanner with ill-formed DTD declaration. Fix by David Bertoni.
+ *
* Revision 1.20 2002/12/24 16:12:19 tng
* For performance reason, move the character check to scancharref.
*
@@ -1268,8 +1271,8 @@
&& (opCh != chCloseParen))
{
// Not a legal char, so delete our node and return failure
- fScanner->emitError(XMLErrs::ExpectedSeqChoiceLeaf);
delete curNode;
+ fScanner->emitError(XMLErrs::ExpectedSeqChoiceLeaf);
return 0;
}
@@ -3303,8 +3306,8 @@
// Has to be the closing paren now.
if (!fReaderMgr->skippedChar(chCloseParen))
{
- fScanner->emitError(XMLErrs::UnterminatedContentModel);
delete headNode;
+ fScanner->emitError(XMLErrs::UnterminatedContentModel);
return false;
}
@@ -3344,8 +3347,8 @@
// Get a name token
if (!fReaderMgr->getName(nameBuf))
{
- fScanner->emitError(XMLErrs::ExpectedElementName);
delete headNode;
+ fScanner->emitError(XMLErrs::ExpectedElementName);
return false;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-cvs-help@xml.apache.org