Pluto
  1. Pluto
  2. PLUTO-332

PortletContextManager.remove(InternalPortletContext) fails

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.1
    • Fix Version/s: 1.1.2
    • Component/s: portlet container
    • Labels:
      None

      Description

      The destroy() method of the PortletServlet throws an exception when shutting down the container:

      ERROR [ContainerBackgroundProcessor[StandardEngine[Catalina]]] [localhost].[/uP3] Mar/05 14:44:41.832 - Servlet GenericXsltPortlet threw unload() exception
      javax.servlet.ServletException: Servlet.destroy() for servlet GenericXsltPortlet threw exception
      at org.apache.catalina.core.StandardWrapper.unload(StandardWrapper.java:1339)
      at org.apache.catalina.core.StandardWrapper.stop(StandardWrapper.java:1654)
      at org.apache.catalina.core.StandardContext.stop(StandardContext.java:4345)
      at org.apache.catalina.startup.HostConfig.checkResources(HostConfig.java:1081)
      at org.apache.catalina.startup.HostConfig.check(HostConfig.java:1187)
      at org.apache.catalina.startup.HostConfig.lifecycleEvent(HostConfig.java:292)
      at org.apache.catalina.util.LifecycleSupport.fireLifecycleEvent(LifecycleSupport.java:119)
      at org.apache.catalina.core.ContainerBase.backgroundProcess(ContainerBase.java:1305)
      at org.apache.catalina.core.ContainerBase$ContainerBackgroundProcessor.processChildren(ContainerBase.java:1569)
      at org.apache.catalina.core.ContainerBase$ContainerBackgroundProcessor.processChildren(ContainerBase.java:1578)
      at org.apache.catalina.core.ContainerBase$ContainerBackgroundProcessor.run(ContainerBase.java:1558)
      at java.lang.Thread.run(Thread.java:595)

      Tracing the code execution this is tracked to the PortletContextManager.remove(InternalPortletContext) call in which a Map keySet Iterator is used but remove() is called on the Map. Instead of calling remove() on the Map it should be called on the Iterator in use to prevent the ConcurrentModificationException.

        Activity

        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12565288 ] jira [ 12586067 ]
        Mark Thomas made changes -
        Workflow jira [ 12398790 ] Default workflow, editable Closed status [ 12565288 ]
        Elliot Metsger made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        David DeWolf made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        David DeWolf made changes -
        Field Original Value New Value
        Fix Version/s 1.1.2 [ 12312360 ]
        Assignee David DeWolf [ ddewolf ]
        Eric Dalquist created issue -

          People

          • Assignee:
            David DeWolf
            Reporter:
            Eric Dalquist
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development