Jetty

ConcurrentModificationException in AbstractSessionManager

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 6.1.5
  • Fix Version/s: 6.1.6rc1
  • Component/s: Servlet
  • Labels:
    None
  • Number of attachments :
    0

Description

Reported by Silvio Bierman on the jetty lists:

Hello all,

I got the CME in 6.1.5 this time:

Exception in thread "Timer-13" java.util.ConcurrentModificationException
at java.util.HashMap$HashIterator.nextEntry(Unknown Source)
at java.util.HashMap$KeyIterator.next(Unknown Source)
at
org.mortbay.jetty.servlet.AbstractSessionManager$Session.doInvalidate(AbstractSessionManager.java:954)
at
org.mortbay.jetty.servlet.AbstractSessionManager$Session.timeout(AbstractSessionManager.java:924)
at
org.mortbay.jetty.servlet.HashSessionManager.scavenge(HashSessionManager.java:200)
at
org.mortbay.jetty.servlet.HashSessionManager.access$000(HashSessionManager.java:36)
at
org.mortbay.jetty.servlet.HashSessionManager$1.run(HashSessionManager.java:144)
at java.util.TimerThread.mainLoop(Unknown Source)
at java.util.TimerThread.run(Unknown Source)

Activity

Ascending order - Click to sort in descending order
Greg Wilkins made changes -
Field Original Value New Value
Assignee Jan Bartel [ janb ] Greg Wilkins [ gregw ]
Hide
Permalink
Greg Wilkins added a comment -

I think this could only have happened if you had attribute listeners that modified other attributes?

I have refactored this method so this should be OK now, but I did not have a test case, so if you could test this
manually, that would be great.

Show
Greg Wilkins added a comment - I think this could only have happened if you had attribute listeners that modified other attributes? I have refactored this method so this should be OK now, but I did not have a test case, so if you could test this manually, that would be great.
Greg Wilkins made changes -
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 6.1.6rc1 [ 13653 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Greg Wilkins
    Reporter:
    Jan Bartel
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: