You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@commons.apache.org by bu...@apache.org on 2005/07/15 15:16:35 UTC

DO NOT REPLY [Bug 35756] New: - PADDING array in StringUtils overflows on '\uffff'

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35756>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35756

           Summary: PADDING array in StringUtils overflows on '\uffff'
           Product: Commons
           Version: 2.1 Final
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Lang
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: sw@reubensivan.com


The PADDING array in class StringUtils is one entry too short.
As declared, it overflows on '\uffff'.
To test, run:
 StringUtils.leftPad("abcd", 10, '\ufffe') // no exception
 StringUtils.leftPad("abcd", 10, '\uffff') // ArrayIndexOutOfBoundsException

Current code:
    private static final String[] PADDING = new String[Character.MAX_VALUE];
Suggested fix: 
    private static final String[] PADDING = new String[Character.MAX_VALUE + 1];

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org