Log

Author Commit Date CI Message
Fabien Degomme b40c1940 2015-10-23T19:35:02 Fix a small error in xmllint --format description Obviously it operates on the output not the input
Daniel Veillard bd0526e6 2015-10-23T19:02:28 Another variation of overflow in Conditional sections Which happen after the previous fix to https://bugzilla.gnome.org/show_bug.cgi?id=756456 But stopping the parser and exiting we didn't pop the intermediary entities and doing the SKIP there applies on an input which may be too small
Gaurav Gupta cf77e605 2015-09-30T14:46:29 Add missing Null check in xmlParseExternalEntityPrivate For https://bugzilla.gnome.org/show_bug.cgi?id=755857 a case where we check for NULL but not everywhere
Daniel Veillard 4a5d80ad 2015-09-18T15:06:46 Fix a bug in CData error handling in the push parser For https://bugzilla.gnome.org/show_bug.cgi?id=754947 The checking function was returning incorrect args in some cases Adds the test to teh reg suite and fix one of the existing test output
Daniel Veillard 51f02b0a 2015-09-15T16:50:32 Fix a bug on name parsing at the end of current input buffer For https://bugzilla.gnome.org/show_bug.cgi?id=754946 When hitting the end of the current input buffer while parsing a name we could end up loosing the beginning of the name, which led to various issues.
Daniel Veillard ef709ce2 2015-09-10T19:41:41 Fix the spurious ID already defined error For https://bugzilla.gnome.org/show_bug.cgi?id=737840 the fix for 724903 introduced a regression on external entities carrying IDs, revert that patch in part and add a specific test to avoid readding it
Nick Wellnhofer 3eaedba1 2015-07-11T14:27:34 Fix previous change to node sort order Commit ba58f23 broke comparison of nodes from different documents. Thanks to Olli Pottonen for the report.
Daniel Veillard 73b2d73d 2015-07-03T20:58:33 Avoid XSS on the search of xmlsoft.org query string need to be escaped before being displayed back
Daniel Veillard 140c251e 2015-06-30T11:36:28 Recover unescaped less-than character in HTML recovery parsing As pointed by Christian Schoenebeck <schoenebeck@crudebyte.com> on the list and based on some of his early patches, this preserve content when unescaped opening angle brackets are not escaped in textual content like: <p> a < b </p> <p> a <0 </p> <p> a <=0 </p> while still reporting the error.
Scott Graham 58b84e1f 2015-06-30T10:47:16 Fix a self assignment issue raised by clang For https://bugzilla.gnome.org/show_bug.cgi?id=751679 Also added a few newline cleanups
Daniel Veillard 709a9521 2015-06-29T16:10:26 Fail parsing early on if encoding conversion failed For https://bugzilla.gnome.org/show_bug.cgi?id=751631 If we fail conversing the current input stream while processing the encoding declaration of the XMLDecl then it's safer to just abort there and not try to report further errors.
Daniel Veillard 9aa37588 2015-06-29T09:08:25 Do not process encoding values if the declaration if broken For https://bugzilla.gnome.org/show_bug.cgi?id=751603 If the string is not properly terminated do not try to convert to the given encoding.
Michael Catanzaro b02a167a 2015-04-14T13:51:01 Silence clang's -Wunknown-attribute Clang doesn't have perfect feature compatibility with GCC, unfortunately. https://bugzilla.gnome.org/show_bug.cgi?id=747870
Patrick Monnerat 22b0415a 2015-04-16T16:04:06 os400: fix various ILE/RPG types definitions. Adjust build scripts. - A typo caused an undefined symbol reference. - A structure field name did not match the corresponding C name due to a typo. - Some structured fields were not properly aligned. - The long/ulong types were wrongly mapped to 64-bit types. - A typo in a /include directive caused a compilation error. - Doc files copy now converts from UTF-8 and split long lines. - Adjust /include file name mapping translation for proper prefix handling.
Daniel Veillard 213f1fe0 2015-04-14T17:41:48 CVE-2015-1819 Enforce the reader to run in constant memory One of the operation on the reader could resolve entities leading to the classic expansion issue. Make sure the buffer used for xmlreader operation is bounded. Introduce a new allocation type for the buffers for this effect.
Martin von Gagern 8985cde7 2015-04-13T16:32:14 xmlMemUsed is not thread-safe For https://bugzilla.gnome.org/show_bug.cgi?id=747437 just use the mutex to protect access to those variables
Shaun McCance 7607d9dd 2015-04-03T22:52:36 Allow HTML serializer to output HTML5 DOCTYPE For https://bugzilla.gnome.org/show_bug.cgi?id=747301 Use simple HTML5 DOCTYPE for about:legacy-compat HTML5 uses a DOCTYPE without a PUBLIC or SYSTEM identifier. It looks like this: <!DOCTYPE html> I can't use XSLT to output this, because to get a DOCTYPE I have to provide a PUBLIC or SYSTEM identifier. Luckily, the standards folks recognized this and provided this semantically equivalent form for the HTML DOCTYPE: <!DOCTYPE html SYSTEM "about:legacy-compat"> But people don't like seeing the "legacy" identifier in their output. They'd rather see the shiny new DOCTYPE. Since we know that about:legacy-compat is defined by the W3C to be semantically equivalent to the sans-SYSTEM DOCTYPE, we could just special-case it in the HTML serializer in libxml2. So if you set the SYSTEM identifier to "about:legacy-compat", you get an HTML5 short-form DOCTYPE.
Daniel Veillard 2fab235d 2015-03-16T08:38:36 Fix support for except in nameclasses For https://bugzilla.gnome.org/show_bug.cgi?id=565219 The code was imply missing even if simple, added a few regression tests.
Nick Wellnhofer 02b252d7 2015-03-08T17:00:37 Regression test for bug #695699
Nick Wellnhofer 342658a1 2015-03-08T16:46:04 Add a couple of XPath tests
Nick Wellnhofer ba58f23c 2015-03-08T16:44:11 Fix order of root nodes Make sure root nodes are sorted before other nodes.
Nick Wellnhofer f6aaabce 2015-03-08T16:05:26 Allow attributes on descendant-or-self axis If the context node is an attribute, the attribute itself is on the descendant-or-self axis. The principal node type of this axis is element, so the only node test that can return the attribute is "node()". In other words, "@attr/descendant-or-self::node()" is equivalent to "@attr". This matches the behavior of Saxon-CE.
Steve Nairn 620a7061 2015-03-03T19:40:06 Fix the fix to Windows locking For https://bugzilla.gnome.org/show_bug.cgi?id=737851 Unfortunately this change has introduced a problem which results in occasional hangs on Windows when running multi-threaded on a multi-core host. When locking the xmlRMutex the count field is increment inside the critical section but when unlocking the count field is decremented outside the critical section. The increment/decrement is not atomic so this can result in the count field being updated incorrectly. The solution is to change xmlRMutexUnlock to decrement the count field before leaving the critical section rather than after.
Christopher Swenson 9b987f8c 2015-02-27T14:55:49 Fix timsort invariant loop re: Envisage article See http://envisage-project.eu/proving-android-java-and-python-sorting-algorithm-is-broken-and-how-to-fix-it/ We use a "runLen" array of size 128, so it should be nearly impossible to have our implementation overflow. But in any case, the fix is relatively simple -- checking two extra conditions in the invariant calculation. I also took this opportunity to remove some redundancy in the left/right merge logic in the invariant loop.
Daniel Veillard 9b851233 2015-02-23T11:29:20 Cleanup conditional section error handling For https://bugzilla.gnome.org/show_bug.cgi?id=744980 The error handling of Conditional Section also need to be straightened as the structure of the document can't be guessed on a failure there and it's better to stop parsing as further errors are likely to be irrelevant.
Daniel Veillard a7dfab74 2015-02-23T11:17:35 Stop parsing on entities boundaries errors For https://bugzilla.gnome.org/show_bug.cgi?id=744980 There are times, like on unterminated entities that it's preferable to stop parsing, even if that means less error reporting. Entities are feeding the parser on further processing, and if they are ill defined then it's possible to get the parser to bug. Also do the same on Conditional Sections if the input is broken, as the structure of the document can't be guessed.
Tomas Radej fb8e9e21 2015-02-18T22:47:13 Add Python 3 rpm subpackage Allow compilation and packaging of both python2 and python3 when present
Samuel Martin 32119739 2014-12-29T20:44:06 libxml2-config.cmake.in: update include directories Align the include directories on those from the pkg-config module. Signed-off-by: Samuel Martin <s.martin49@gmail.com>
Nick Wellnhofer 220a7bae 2014-12-23T21:28:37 Don't add IDs in xmlSetTreeDoc This partially reverts my previous commit fixing bug #741919.
Michael Heimpold fff8a6b8 2014-12-22T11:12:12 threads: use forward declarations only for glibc Fixes bug #704908 The declarations of pthread functions, used to generate weak references to them, fail to suppress macros. Thus, if any pthread function has been provided as a macro, compiling threads.c will fail. This breaks on musl libc, which defines pthread_equal as a macro (in addition to providing the function, as required). Prevent the declarations for e.g. musl libc by refining the condition. The idea for this solution was borrowed from the alpine linux guys, see http://git.alpinelinux.org/cgit/aports/tree/main/libxml2/libxml2-pthread.patch Signed-off-by: Michael Heimpold <mhei@heimpold.de>
Nick Wellnhofer f54d6a92 2014-12-19T00:08:35 Account for ID attributes in xmlSetTreeDoc
Daniel Veillard 7580ce0a 2014-10-28T18:13:53 Update Win32 configure.js to search for configure.ac as configure.in was renamed
Philip Withnall 57941042 2014-10-26T18:08:04 Remove various unused value assignments As detected by Coverity (CIDs 60467–60472). https://bugzilla.gnome.org/show_bug.cgi?id=739220
Daniel Veillard df23f584 2014-10-23T13:52:47 Adding example from bugs 738805 to regression tests For https://bugzilla.gnome.org/show_bug.cgi?id=738805 Tortuous test case provided by pierre.labastie@neuf.fr
Daniel Veillard 72a46a51 2014-10-23T11:35:36 Fix missing entities after CVE-2014-3660 fix For https://bugzilla.gnome.org/show_bug.cgi?id=738805 The fix for CVE-2014-3660 introduced a regression in some case where entity substitution is required and the entity is used first in anotther entity referenced from an attribute value
Daniel Veillard f65128f3 2014-10-17T17:13:41 Revert "Missing initialization for the catalog module" This reverts commit 054c716ea1bf001544127a4ab4f4346d1b9947e7. As this break xmlcatalog command https://bugzilla.redhat.com/show_bug.cgi?id=1153753
Daniel Veillard 726f67e2 2014-10-16T15:40:16 Release of libxml2-2.9.2
Daniel Veillard 2fcdb424 2014-10-16T15:39:41 Forgot to add the cmake Makefile
Daniel Veillard be2a7eda 2014-10-16T13:59:47 Fix for CVE-2014-3660 Issues related to the billion laugh entity expansion which happened to escape the initial set of fixes
Bart De Schuymer 500c54ef 2014-10-16T12:17:20 fix memory leak xml header encoding field with XML_PARSE_IGNORE_ENC When the xml parser encounters an xml encoding in an xml header while configured with option XML_PARSE_IGNORE_ENC, it fails to free memory allocated for storing the encoding. The patch below fixes this. How to reproduce: 1. Change doc/examples/parse4.c to add xmlCtxtUseOptions(ctxt, XML_PARSE_IGNORE_ENC); after the call to xmlCreatePushParserCtxt. 2. Rebuild 3. run the following command from the top libxml2 directory: LD_LIBRARY_PATH=.libs/ valgrind --leak-check=full ./doc/examples/.libs/parse4 ./test.xml , where test.xml contains following input: <?xml version="1.0" encoding="UTF-81" ?><hi/> valgrind will report: ==1964== 10 bytes in 1 blocks are definitely lost in loss record 1 of 1 ==1964== at 0x4C272DB: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) ==1964== by 0x4E88497: xmlParseEncName (parser.c:10224) ==1964== by 0x4E888FE: xmlParseEncodingDecl (parser.c:10295) ==1964== by 0x4E89630: xmlParseXMLDecl (parser.c:10534) ==1964== by 0x4E8B737: xmlParseTryOrFinish (parser.c:11293) ==1964== by 0x4E8E775: xmlParseChunk (parser.c:12283) Signed-off-by: Bart De Schuymer <bart at amplidata com>
Roumen Petrov 974db365 2014-10-16T12:07:43 AC_CONFIG_FILES and executable bit
Roumen Petrov f3efaba3 2012-09-19T00:10:55 remove HAVE_CONFIG_H dependency in testlimits.c
Roumen Petrov c2450d8b 2013-11-28T22:42:08 fix some tabs mixing incompatible with python3
Roumen Petrov 21fec499 2012-09-12T01:33:12 win32/libxml2.def.src after rebuild in doc
Roumen Petrov 4bb03d7a 2014-03-23T16:16:49 elfgcchack.h: more legacy needs xmlSAX2StartElement() and xmlSAX2EndElement()
Roumen Petrov c7684f9b 2013-04-13T18:11:01 elfgcchack.h: add xmlXPathNodeEval and xmlXPathSetContextNode (after rebuild in doc)
Yegor Yefremov 7446445d 2014-10-10T12:23:09 xmlmemory: handle realloc properly If realloc fails, free original pointer. Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
Samuel Martin fa23ac1a 2014-10-11T21:50:41 Provide cmake module * add libxml2-config.cmake.in template * configure.ac: add libxml2-config.cmake.in to the configured file list * Makefile.am: install libxml2-config.cmake under ${libdir}/cmake/libxml2 Signed-off-by: Samuel Martin <s.martin49@gmail.com>
Daniel Veillard af62eb4f 2014-10-13T16:40:56 Python generator bug raised by the const change the doc methode for xmlDocGetRootElement wasn't generated anymore
Daniel Veillard 7a72f4af 2014-10-13T16:23:24 Fix a couple of issues raised by make dist
Kurt Roeckx 95ebe53b 2014-10-13T16:06:21 Fix and add const qualifiers For https://bugzilla.gnome.org/show_bug.cgi?id=689483 It seems there are functions that do use the const qualifier for some of the arguments, but it seems that there are a lot of functions that don't use it and probably should. So I created a patch against 2.9.0 that makes as much as possible const in tree.h, and changed other files as needed. There were a lot of cases like "const xmlNodePtr node". This doesn't actually do anything, there the *pointer* is constant not the object it points to. So I changed those to "const xmlNode *node". I also removed some consts, mostly in the Copy functions, because those functions can actually modify the doc or node they copy from
Francis Dupont 12fd6543 2014-10-13T15:09:43 Visual Studio 14 CTP defines snprintf() For https://bugzilla.gnome.org/show_bug.cgi?id=737937 Visual Studio 14 CTP (the VS which comes with Windows 10) defines snprintf(). It could be seen as a good idea as snprintf() is part of the C99 standard but unfortunately libxml2 as many packages defines snprintf as _snprintf, the function to use for any previous versions of the Visual Studio runtime. More, to avoid hiding/shadowing snprintf() declaration in stdio.h is protected by an "#ifdef snprintf" followed by an "#error", so compilation fails. But the fix is easy: the corresponding C/C++ compiler defines _MSC_VER to 1900 so it is enough to guard the snprintf define against it, cf. the attached patch for win32config.h (from 2.9.1 "latest" tarball).
Daniel Veillard 8854e463 2014-10-13T15:03:58 Windows Critical sections not released correctly For https://bugzilla.gnome.org/show_bug.cgi?id=737851 Based on report from mike.vanduzee@caris.com , we were missing calling LeaveCriticalSection() when count was down to 0 hence not freeing adequately the related resource.
Patrick Monnerat 5018082b 2014-10-09T17:39:01 OS400: do not try to copy unexisting doc files
Patrick Monnerat ad160cb3 2014-10-09T16:01:47 OS400: use either configure.ac or configure.in.
Daniel Veillard d0dc5813 2014-10-10T15:35:10 Removal of old configure.in
Daniel Veillard f985ada7 2014-10-09T23:59:36 Preparing for upcoming release of 2.9.2 Moving configure.in to configure.ac since all tools complain about it
Daniel Veillard 7cf57380 2014-10-08T16:09:56 Parser error on repeated recursive entity expansion containing &lt; For https://bugzilla.gnome.org/show_bug.cgi?id=736417 basically a weird side effect and a failure to properly parenthesize a boolean expression led to this bug
Dmitriy 7dc24965 2014-10-07T18:59:00 Fix zlib and lzma libraries check via command line For https://bugzilla.gnome.org/show_bug.cgi?id=737532 add save, change and restore LDFLAGS before AC_CHECK_LIB call when functions gzread and lzma_code are searching inside action-if-found of AC_CHECK_HEADERS
Gaurav Gupta f5496a1e 2014-10-07T17:09:35 xpointer : fixing Null Pointers For https://bugzilla.gnome.org/show_bug.cgi?id=738053 At many places in xpointer.c Null check is missing which is dereferenced at later places.
Gaurav Gupta a61929e8 2014-10-07T12:28:34 Remove Unnecessary Null check in xpointer.c For https://bugzilla.gnome.org/show_bug.cgi?id=738052 superfluous check
Dennis Filder 7e9bbdf8 2014-10-06T20:34:14 parser bug on misformed namespace attributes For https://bugzilla.gnome.org/show_bug.cgi?id=672539 Reported by Axel Miller <axel.miller@ppi.de> Consider the following start-tag: <x xmlns=""version=""> The start-tag does not conform to the rule [40] STag ::= '<' Name (S Attribute)* S? '>' since there is no whitespace in front of the attribute "version". Thus, libxml2 should reject the start-tag. But it doesn't: $ echo '<x xmlns=""version=""/>' | xmllint - <?xml version="1.0"?> <x xmlns="" version=""/> The error seems to happen only if there is a namespace declaration in front of the attribute. A missing whitespace between other attributes is handled correctly: $ echo '<x someattr=""version=""/>' | xmllint - -:1: parser error : attributes construct error <x someattr=""version=""/> ^ [...]
Gaurav Gupta 6d93e9ea 2014-10-06T20:20:00 Unreachable code in tree.c For https://bugzilla.gnome.org/show_bug.cgi?id=705392 Cut out an unused block
Daniel Veillard 91309d3a 2014-10-06T20:07:19 Pointer dereferenced before null check For https://bugzilla.gnome.org/show_bug.cgi?id=707027 A few pointer dereference before NULL check fixed. Removed a useless test
Gaurav Gupta d72cb06d 2014-10-06T19:28:29 Leak of struct addrinfo in xmlNanoFTPConnect() For https://bugzilla.gnome.org/show_bug.cgi?id=732352 in case of error condition in IPv6 support, the early return here doesn't call freeaddrinfo(result), thus leaking memory.
Daniel Veillard 292a9f29 2014-10-06T18:51:04 Possible overflow in HTMLParser.c For https://bugzilla.gnome.org/show_bug.cgi?id=720615 make sure that the encoding string passed is of reasonable size
John Beck 6bea543e 2014-10-06T18:26:27 python/tests/sync.py assumes Python dictionaries are ordered For https://bugzilla.gnome.org/show_bug.cgi?id=734017 Solaris has had libxml2 version 2.9.1 for a while, with Python versions 2.6 and 2.7. While preparing to also build a module for Python 3.4, we ran into an issue with the test case sync.py failing. The failure involved parsing a string that included a Python dictionary, then complaining when the order of the parsed result did not match the original order. But Python dictionaries are unordered by definition; see section 5.5 of https://docs.python.org/2/tutorial/datastructures.html . For whatever reason, Python 2.6 and 2.7 always happened to report the pair of values back in their original order, but with Python 3.4 the order is random. The attached patch allows for either order; it also fixes a typo that was repeated several times thanks to the magic of copy & paste.
Juergen Keil 24fb4c32 2014-10-06T18:19:12 wrong error column in structured error when parsing end tag For https://bugzilla.gnome.org/show_bug.cgi?id=734283 libxml2 reports wrong error column numbers (field int2 in xmlError) in structured error handler, after parsing an end tag.
Gaurav Gupta d319eb92 2014-10-06T12:24:17 Fix Enum check and missing break for https://bugzilla.gnome.org/show_bug.cgi?id=737403 In file xmlreader.c 1. An enum is checked to proper value instead of checking like a boolean. 2. Missing break statement added.
Philip Withnall 21699937 2014-08-24T23:10:13 xmlIO: Handle error returns from dup() If dup() fails and returns -1, gzdopen() will transparently return NULL, but then close() will fail after being called on an invalid FD. Change the code to only call close() on valid FDs. Coverity issue: #72382
Patrick Monnerat 98d71f91 2014-10-03T12:36:36 os400: make-src.sh: create physical file with target CCSID
Patrick Monnerat 5378ff93 2014-05-06T13:39:40 OS400: Add some more C macros equivalent procedures.
Patrick Monnerat bce6a61e 2014-05-06T11:31:16 OS400: use C macros to implement equivalent RPG support procedures.
Patrick Monnerat 02fd1298 2014-05-05T18:16:22 OS400: implement XPath macros as procedures for ILE/RPG support.
Patrick Monnerat 917e353f 2014-03-04T17:36:54 OS400: include in distribution tarball.
Patrick Monnerat c0179883 2014-03-04T17:27:22 OS400: Add README: compilation directives and OS/400 specific stuff.
Patrick Monnerat e72b4e73 2014-03-04T17:26:13 OS400: Add compilation scripts.
Patrick Monnerat e083c30e 2014-03-04T17:18:26 OS400: ILE RPG language header files.
Patrick Monnerat 7d888f07 2014-03-04T17:15:23 OS400: implement some macros as functions for ILE/RPG language support (that as no macros).
Patrick Monnerat 6e4d8703 2014-03-04T17:11:02 OS400: UTF8<-->EBCDIC wrappers for system and external library calls
Patrick Monnerat 5621c81b 2014-03-04T17:09:26 OS400: Easy character transcoding support
Patrick Monnerat 17951ea2 2014-03-04T16:42:19 OS400: iconv functions compatibility wrappers and table builder.
Patrick Monnerat ea8c89b9 2014-03-04T16:13:34 doc/news.html: small update to avoid line join while generating NEWS.
Patrick Monnerat a6c5b022 2014-03-04T15:47:08 OS400: create architecture directory. Implement dlfcn emulation.
Daniel Veillard beb72810 2014-10-03T19:22:39 Fix a problem properly saving URIs As written by Martin Kletzander <mkletzan@redhat.com>: Since commit 8eb55d782a2b9afacc7938694891cc6fad7b42a5, when you parse and save an URI that has no server (or similar) part, two slashes after the 'schema:' get lost. It means 'uri:///noserver' is turned into 'uri:/noserver'. basically foo:///only/path means a host of "" while foo:/only/path means no host at all So the best fix IMHO is to fix the URI parser to record the first case and an empty host string and the second case as a NULL host string I would not revert the initial patch, we should not 'invent' those slash, but we should instead when parsing keep the information that it's a host based path and that foo:/// means the presence of a host but an empty one. Once applied the resulting patch below, all cases seems to be saved properly: thinkpad:~/XML -> ./testURI uri:/noserver uri:/noserver thinkpad:~/XML -> ./testURI uri:///noserver uri:///noserver thinkpad:~/XML -> ./testURI uri://server/foo uri://server/foo thinkpad:~/XML -> ./testURI uri:/noserver/foo uri:/noserver/foo thinkpad:~/XML -> ./testURI uri:/// uri:/// thinkpad:~/XML -> ./testURI uri:// uri:// thinkpad:~/XML -> ./testURI uri:/ uri:/ thinkpad:~/XML -> If you revert the initial patch that last case fails The problem is that I don't want to change the xmlURI structure to minimize ABI breakage, so I could not extend the field. The natural solution is to denote that uri:/// has an empty host by making the uri server field an empty string which works very well but breaks applications (like libvirt ;-) who blindly look at uri->server not being NULL to try to reach it ! Simplest was to stick the port to -1 in that case, instead of 0 application don't bother looking at the port of there is no server string, this makes the patch more complex than a 1 liner, but is better for ABI.
Ron Angeles b3e488b0 2014-09-27T21:56:03 Add methods for python3 iterator xmlCoreDepthFirstItertor and xmlCoreBreadthFirstItertr only implement a python2-compatible iterator interface. The next() method has been changed to __next__(). An alias has been defined to keep python2 compatibility.
Juergen Keil 33f658c9 2014-08-07T17:30:36 wrong error column in structured error when parsing attribute values For https://bugzilla.gnome.org/show_bug.cgi?id=734280 libxml2 reports wrong error column numbers (field int2 in xmlError) in structured error handler, after parsing XML attribute values. Example XML: <?xml version="1.0" encoding="UTF-8"?> <root xmlns="urn:colbug">&</root> <!-- 1 2 3 4 1234567890123456789012345678901234567890 --> Expected location of the error would be line 3, column 21. The actual location of the error is line 3, column 9: $ ./xmlparse colbug2.xml colbug2.xml:3:9: xmlParseEntityRef: no name The 12 characters of the xmlns attribute value "urn:colbug" are not accounted for in the error column value.
Juergen Keil 5d4310af 2014-08-07T16:28:09 wrong error column in structured error when skipping whitespace in xml decl For https://bugzilla.gnome.org/show_bug.cgi?id=734276 libxml2 reports wrong error column numbers (field int2 in xmlError) in structured error handler, after an XML declaration containing whitespace. Example XML: <?xml version="1.0" encoding="UTF-8" ?><root>&</root> <!-- 1 2 3 4 5 6 123456789012345678901234567890123456789012345678901234567890 --> Expected location of the error would be line 1, column 53. The actual location of the error is line 1, column 44: $ ./xmlparse colbug1.xml colbug1.xml:1:44: xmlParseEntityRef: no name
Juergen Keil d201e71e 2014-08-07T11:42:07 no error column in structured error handler for xml schema validation errors For https://bugzilla.gnome.org/show_bug.cgi?id=734363 When using xml schema validation, structured error callbacks do not get passed a valid column number in xmlError field "int2". $ ./xmlsaxparse colbug5.xml colbug5.xsd colbug5.xml:3:0: Element '{urn:colbug5}bx': This element is not expected. Expected is ( {urn:colbug5}b ). The schema error is reported for line 3, column 0 (= N/A). I'd like to have the column number of the error passed in the xmlError structure. With this test case: line 3, column 9.
Gaurav Gupta 658b86c0 2014-08-07T11:19:03 Couple of Missing Null checks For https://bugzilla.gnome.org/show_bug.cgi?id=734328 Missing Null check could cause crash, if a pointer is dereferenced. Found problem at two places in valid.c
Kyle VanderBeek 1db99699 2014-07-29T00:32:15 Support element node traversal in document fragments. https://bugzilla.gnome.org/show_bug.cgi?id=733900
Gaurav Gupta b8480ae7 2014-07-26T21:14:53 Remove a couple of dead conditions For https://bugzilla.gnome.org/show_bug.cgi?id=733711
Daniel Veillard 42870f46 2014-07-26T21:04:54 Add couple of missing Null checks For https://bugzilla.gnome.org/show_bug.cgi?id=733710 Reported by Gaurav but with slightly different fixes
Daniel Veillard 2f9b126a 2014-07-26T20:29:36 typo in error messages "colon are forbidden from..." For https://bugzilla.gnome.org/show_bug.cgi?id=731511 Pointed byt vincent Lefevre
Philip Withnall 4ba5d317 2014-06-20T21:37:21 xmlschemastypes: Fix potential array overflow The year and month need validating before being put into the MAX_DAYINMONTH macro. Coverity issue: #60436 https://bugzilla.gnome.org/show_bug.cgi?id=731990
Philip Withnall 5777ae75 2014-06-20T21:15:16 runtest: Fix a memory leak on parse failure Coverity issue: #60439 https://bugzilla.gnome.org/show_bug.cgi?id=731990
Philip Withnall 31aa3815 2014-06-20T21:11:40 xmlIO: Fix an FD leak on gzdopen() failure According to the documentation, gzdopen() does not close the FD on failure (but does effectively close it on success, since gzclose() closes it). Coverity issues: #60440, #60441 https://bugzilla.gnome.org/show_bug.cgi?id=731990
Philip Withnall 7746f2f6 2014-06-20T21:05:33 xmlcatalog: Fix a memory leak on quit Coverity issue: #60442 https://bugzilla.gnome.org/show_bug.cgi?id=731990
Philip Withnall 579ebbcb 2014-06-20T21:03:42 HTMLparser: Correctly initialise a stack allocated structure If not initialised, the ‘node’ member remains undefined. Coverity issue: #60466 https://bugzilla.gnome.org/show_bug.cgi?id=731990
Daniel Veillard 319e159b 2014-07-15T11:13:15 Fix building when configuring without xpath and xptr For https://bugzilla.gnome.org/show_bug.cgi?id=732735 schematron little used code and xptr rely on XPath, fix the configure script.