ICEfaces
  1. ICEfaces
  2. ICE-4687

An exception during Request.respondWith(...) causes ThreadBlockingAdaptingServlet's semaphore to not be released.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.8.1
    • Fix Version/s: 1.8.2-RC1, 1.8.2
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces Core, ICEfaces Push Server

      Description

      ICEfaces application bugs exposed a thread leak in the Thread Blocking environment. If an exception is thrown while trying to send a response to a request the Servlet Container supplied thread got blocked and never got unblocked. The Thread Blocking environment relies on semaphore logic to block and unblock the Servlet Container supplied thread. This thread tries to acquire the semaphore if no response is available causing this thread to block. When a response becomes available another thread releases the semaphore in order to unblock the Servlet Container supplied thread. If an exception occurred during this process the semaphore was not released.

      Basically, an exception during Request.respondWith(...) causes ThreadBlockingAdaptingServlet's semaphore to not be released:

          public void respondWith(ResponseHandler handler) throws Exception {
              super.respondWith(handler);
              if (semaphore == null) {
                  blockResponse = false;
              } else {
                  semaphore.release();
              }
          }

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #19049 Thu Jul 09 13:21:23 MDT 2009 jack.van.ooststroom Fixed JIRA ICE-4687 : An exception during Request.respondWith(...) causes ThreadBlockingAdaptingServlet's semaphore to not be released.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/ThreadBlockingAdaptingServlet.java

          People

          • Assignee:
            Jack Van Ooststroom
            Reporter:
            Jack Van Ooststroom
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: