You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Lukas Theussl (JIRA)" <ji...@codehaus.org> on 2010/05/22 09:52:12 UTC
[jira] Closed: (DOXIA-370) Confluence module cannot parse
horizontal separator
[ http://jira.codehaus.org/browse/DOXIA-370?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Lukas Theussl closed DOXIA-370.
-------------------------------
Resolution: Fixed
Fix Version/s: 1.1.3
Assignee: Lukas Theussl
Patch applied in [r947266|http://svn.apache.org/viewvc?view=revision&revision=947266]. Thanks!
> Confluence module cannot parse horizontal separator
> ---------------------------------------------------
>
> Key: DOXIA-370
> URL: http://jira.codehaus.org/browse/DOXIA-370
> Project: Maven Doxia
> Issue Type: Bug
> Components: Module - Confluence
> Affects Versions: 1.1.1
> Reporter: Dave Syer
> Assignee: Lukas Theussl
> Fix For: 1.1.3
>
> Attachments: DOXIA-370.patch
>
>
> Confluence module cannot parse horizontal separator. E.g. if you parse this
> {code}
> Up
> ----
> Down
> {code}
> you get out of bounds exception in the ListBlockParser. This code looks pretty flakey (I probably wrote it, so apologies if it was my fault):
> {code}
> private boolean isList( String line )
> {
> line = line.trim();
> if ( line.startsWith( "*" ) || line.startsWith( "-" ) || line.startsWith( "#" ) )
> {
> String temp = line.substring( 1 );
> while ( temp.charAt( 0 ) == '*' || temp.charAt( 0 ) == '-' || temp.charAt( 0 ) == '#' )
> {
> temp = temp.substring( 1 );
> }
> if ( temp.charAt( 0 ) == ' ' )
> {
> return true;
> }
> }
> return false;
> }
> {code}
> There are a load of potential out of bounds exceptions there. This one happens to come from the fact that the loop never terminates before the temp string is exhausted.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira