Author: ate
Date: Mon Jun 11 13:34:49 2007
New Revision: 546262

URL: http://svn.apache.org/viewvc?view=rev&rev=546262
Log:
Fix potential infinite loop when a RENDER_CONTEXT path transferred from an 
processAction results in another Struts (render) include call.
A RENDER_CONTEXT only needs to be used once.

Modified:
    
portals/bridges/trunk/struts/src/java/org/apache/portals/bridges/struts/PortletServlet.java

Modified: 
portals/bridges/trunk/struts/src/java/org/apache/portals/bridges/struts/PortletServlet.java
URL: 
http://svn.apache.org/viewvc/portals/bridges/trunk/struts/src/java/org/apache/portals/bridges/struts/PortletServlet.java?view=diff&rev=546262&r1=546261&r2=546262
==============================================================================
--- 
portals/bridges/trunk/struts/src/java/org/apache/portals/bridges/struts/PortletServlet.java
 (original)
+++ 
portals/bridges/trunk/struts/src/java/org/apache/portals/bridges/struts/PortletServlet.java
 Mon Jun 11 13:34:49 2007
@@ -133,6 +133,10 @@
                
             if (context != null)
             {
+                // only use the RENDER_CONTEXT once: if the target path is an 
Struts Action which might result in a subsequent
+                // dispatch, the normal Struts processing should happen, 
otherwise a infinite loop might occurr.
+                request.removeAttribute(StrutsPortlet.RENDER_CONTEXT);
+
                 if (log.isDebugEnabled())
                 {
                     log.debug("render context path: " + context.getPath());



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to