You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@xalan.apache.org by "Brian Minchau (JIRA)" <xa...@xml.apache.org> on 2005/07/12 05:26:20 UTC
[jira] Commented: (XALANJ-2154) [XSLTC] Parser: NullPointerException when _locator is null
[ http://issues.apache.org/jira/browse/XALANJ-2154?page=comments#action_12315533 ]
Brian Minchau commented on XALANJ-2154:
---------------------------------------
Andreas,
the code change was clearly to add a bit more location information about where the literal element is in the stylesheet.
For what ever reason _locator is null. It would be easy to add that check
before calling node.setLineNumber(_locator.getLineNumber());
A testcase would be nice, but this one seems trivial enough to fix.
> [XSLTC] Parser: NullPointerException when _locator is null
> ----------------------------------------------------------
>
> Key: XALANJ-2154
> URL: http://issues.apache.org/jira/browse/XALANJ-2154
> Project: XalanJ2
> Type: Bug
> Components: XSLTC
> Reporter: Andreas Hartmann
>
> The exception is caused by the following change (it works with Xalan 2.6.0):
> http://cvs.apache.org/viewcvs.cgi/xml-xalan/java/src/org/apache/xalan/xsltc/compiler/Parser.java?r1=1.64&r2=1.65&diff_format=h
> ===================================================================
> RCS file: /home/cvspublic/xml-xalan/java/src/org/apache/xalan/xsltc/compiler/Parser.java,v
> retrieving revision 1.64
> retrieving revision 1.65
> diff -u -r1.64 -r1.65
> --- Parser.java 2004/02/23 10:29:35 1.64
> +++ Parser.java 2004/05/21 20:29:28 1.65
> @@ -14,7 +14,7 @@
> * limitations under the License.
> */
> /*
> - * $Id: Parser.java,v 1.64 2004/02/23 10:29:35 aruny Exp $
> + * $Id: Parser.java,v 1.65 2004/05/21 20:29:28 santiagopg Exp $
> */
>
> package org.apache.xalan.xsltc.compiler;
> @@ -965,7 +965,10 @@
> }
> }
> }
> - if (node == null) node = new LiteralElement();
> + if (node == null) {
> + node = new LiteralElement();
> + node.setLineNumber(_locator.getLineNumber());
> + }
> }
> if ((node != null) && (node instanceof LiteralElement)) {
> ((LiteralElement)node).setQName(qname);
> +++++++++++++++++++++++++
> Unfortunately I'm not familiar enough with the code to know if a "_locator != null" check would be sufficient.
> Stack trace:
> org.apache.xalan.xsltc.TransletException: java.lang.NullPointerException
> at org.apache.xalan.xsltc.dom.SAXImpl.shallowCopy(SAXImpl.java:1735)
> at org.apache.xalan.xsltc.dom.DOMAdapter.shallowCopy(DOMAdapter.java:314)
> at page2xslt.applyTemplates()
> at page2xslt.applyTemplates()
> at page2xslt.applyTemplates()
> at page2xslt.applyTemplates()
> at page2xslt.applyTemplates()
> at page2xslt.transform()
> at org.apache.xalan.xsltc.runtime.AbstractTranslet.transform(AbstractTranslet.java:592)
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-dev-help@xml.apache.org