Skip to content

Commit

Permalink
Fix crash with unexpected DTD nodes in XSLT.
Browse files Browse the repository at this point in the history
BUG=127417
Review URL: https://chromiumcodereview.appspot.com/10441148

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140041 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
cevans@chromium.org committed Jun 1, 2012
1 parent 05a5c06 commit bb7bfb8
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion third_party/libxslt/README.chromium
Expand Up @@ -40,7 +40,8 @@ done ---

Current version: 1.1.26, plus the following patches:
- A fix to get more compact generated IDs (http://git.gnome.org/browse/libxslt/commit/?id=ecb6bcb8d1b7e44842edde3929f412d46b40c89f)
- Import pattern parsing fix for commit: http://git.gnome.org/browse/libxslt/commit/?id=fe5a4fa33eb85bce3253ed3742b1ea6c4b59b41b
- Import pattern parsing fix for commit: http://git.gnome.org/browse/libxslt/commit/?id=fe5a4fa33eb85bce3253ed3742b1ea6c4b59b41b
- A fix for XSLT node checking (from upstream, commit pending).


To import a new snapshot of libxslt:
Expand Down
4 changes: 2 additions & 2 deletions third_party/libxslt/libxslt/xsltutils.h
Expand Up @@ -52,8 +52,8 @@ extern "C" {
* Checks that the element pertains to XSLT namespace.
*/
#define IS_XSLT_ELEM(n) \
(((n) != NULL) && ((n)->ns != NULL) && \
(xmlStrEqual((n)->ns->href, XSLT_NAMESPACE)))
(((n) != NULL) && ((n)->type == XML_ELEMENT_NODE) && \
((n)->ns != NULL) && (xmlStrEqual((n)->ns->href, XSLT_NAMESPACE)))

/**
* IS_XSLT_NAME:
Expand Down

0 comments on commit bb7bfb8

Please sign in to comment.