[
https://issues.apache.org/jira/browse/AXIS2-5078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13054929#comment-13054929
]
Hudson commented on AXIS2-5078:
-------------------------------
Integrated in Axis2 #820 (See [https://builds.apache.org/job/Axis2/820/])
AXIS2-5078: Fixed an issue in QueryStringParser.
veithen :
Files :
*
/axis/axis2/java/core/trunk/modules/kernel/test/org/apache/axis2/transport/http/util/QueryStringParserTest.java
*
/axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/transport/http/util/QueryStringParser.java
> Axis takes 100% CPU, infinite loop in the QueryStringParser
> -----------------------------------------------------------
>
> Key: AXIS2-5078
> URL: https://issues.apache.org/jira/browse/AXIS2-5078
> Project: Axis2
> Issue Type: Bug
> Components: kernel
> Affects Versions: 1.5.5
> Environment: Mac, Linux
> Reporter: Jonathan Ducharme
> Assignee: Andreas Veithen
> Fix For: 1.5.6, 1.6.1, 1.7.0
>
>
> Hi,
> My problem is basically https://issues.apache.org/jira/browse/AXIS2-4736, but
> since it was closed without the step to reproduce, I've open a new issue, and
> will give you the code I made to test it.
> Yes it's a malformed URL, but I think it's still an issue.
> public static void main( String[] args ) {
> QueryStringParser parser = new QueryStringParser("&gsdfgfds");
> parser.search( new ArrayList<String>() );
> System.out.println("done");
> }
> That being said, what I did is change the next() method by adding a check on
> the idx:
> public boolean next() {
> int len = queryString.length();
> if (paramEnd == len) {
> return false;
> }
> paramBegin = paramEnd == 0 ? 0 : paramEnd+1;
> int idx = queryString.indexOf('&', paramBegin);
>
> if(idx == 0) {
> return false;
> }
>
> paramEnd = idx == -1 ? len : idx;
> idx = queryString.indexOf('=', paramBegin);
> paramNameEnd = idx == -1 || idx > paramEnd ? paramEnd : idx;
> paramName = null;
> paramValue = null;
> return true;
> }
> For our current needs it work, might not be the best solution, but for what
> we had to do it worked pretty well. This also pass the QueryStringParserTest
> that axis has.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
