You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by pb...@apache.org on 2007/04/18 14:35:37 UTC
svn commit: r530001 -
/struts/struts1/trunk/core/src/main/java/org/apache/struts/config/ActionConfigMatcher.java
Author: pbenedict
Date: Wed Apr 18 05:35:36 2007
New Revision: 530001
URL: http://svn.apache.org/viewvc?view=rev&rev=530001
Log:
STR-3029: Apply patch
Modified:
struts/struts1/trunk/core/src/main/java/org/apache/struts/config/ActionConfigMatcher.java
Modified: struts/struts1/trunk/core/src/main/java/org/apache/struts/config/ActionConfigMatcher.java
URL: http://svn.apache.org/viewvc/struts/struts1/trunk/core/src/main/java/org/apache/struts/config/ActionConfigMatcher.java?view=diff&rev=530001&r1=530000&r2=530001
==============================================================================
--- struts/struts1/trunk/core/src/main/java/org/apache/struts/config/ActionConfigMatcher.java (original)
+++ struts/struts1/trunk/core/src/main/java/org/apache/struts/config/ActionConfigMatcher.java Wed Apr 18 05:35:36 2007
@@ -180,7 +180,13 @@
ForwardConfig cfg;
for (int x = 0; x < fConfigs.length; x++) {
- cfg = new ActionForward();
+ try {
+ cfg = (ActionForward) BeanUtils.cloneBean(fConfigs[x]);
+ } catch (Exception ex) {
+ log.warn("Unable to clone action config, recommend not using "
+ + "wildcards", ex);
+ return null;
+ }
cfg.setName(fConfigs[x].getName());
cfg.setPath(convertParam(fConfigs[x].getPath(), vars));
cfg.setRedirect(fConfigs[x].getRedirect());