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

Posted to dev@struts.apache.org by bu...@apache.org on 2004/01/05 19:07:15 UTC

DO NOT REPLY [Bug 25909] New: - ActionConfigMatcher.convertActionConfig() throws an IndexOutOfBoundsException

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

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25909

ActionConfigMatcher.convertActionConfig() throws an IndexOutOfBoundsException

           Summary: ActionConfigMatcher.convertActionConfig() throws an
                    IndexOutOfBoundsException
           Product: Struts
           Version: Nightly Build
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Controller
        AssignedTo: struts-dev@jakarta.apache.org
        ReportedBy: roberto.tyley@reuters.com


The following first line of code was causing an IndexOutOfBoundsException in
org.apache.struts.config.ActionConfigMatcher.convertActionConfig(String path,
ActionConfig orig, Map vars) at line 185, when the variable 'path' was the empty
string:

---
if (path.charAt(0) != '/') {
    path = "/" + path;
}
---

It *is* possible for the 'path' variable to be the empty string, and indeed
ActionConfigMatcher codes defensively for this case elsewhere- just not here.
This bug is present in all versions, including the most recent, of the
ActionConfigMatcher class, as found at:

http://cvs.apache.org/viewcvs.cgi/jakarta-struts/src/share/org/apache/struts/config/ActionConfigMatcher.java

I will attach a patch shortly!

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