ICEfaces
  1. ICEfaces
  2. ICE-8549

Thread-safe member access in MainServlet

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-1.8.2.GA_P03
          • Fix Version/s: EE-1.8.2.GA_P05
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            ICEfaces
          • Assignee Priority:
            P2
          • Affects:
            Compatibility/Configuration

            Description


            The following lines in MainServlet are not thread-safe:

            MainServlet.java:185 localAddress
            MainServlet.java:186 localPort
            MainServlet.java:298 coreMessageService
            MainServlet.java:311 coreMessageService
            MainServlet.java:329 coreMessageService

            Thread-safe getter/setter functions for these fields should be added.

              Activity

              Ascending order - Click to sort in descending order
              Ted Goddard created issue -
              Ted Goddard made changes -
              Field Original Value New Value
              Salesforce Case []
              Arran Mccullough made changes -
              Salesforce Case [5007000000O8ixC]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #31367 Tue Oct 09 14:23:54 MDT 2012 jack.van.ooststroom Fixed JIRA ICE-8549 : Thread-safe member access in MainServlet
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/servlet/MainServlet.java
              Hide
              Permalink
              Jack Van Ooststroom added a comment - - edited

              Sending src/com/icesoft/faces/webapp/http/servlet/MainServlet.java
              Transmitting file data .
              Committed revision 31367.

              Show
              Jack Van Ooststroom added a comment - - edited Sending src/com/icesoft/faces/webapp/http/servlet/MainServlet.java Transmitting file data . Committed revision 31367.
              Hide
              Permalink
              Jack Van Ooststroom added a comment - - edited

              Added a private synchronized method for setting the localAddress and localPort, and added the synchronized keyword to the setUpCoreMessageService() method. Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - - edited Added a private synchronized method for setting the localAddress and localPort, and added the synchronized keyword to the setUpCoreMessageService() method. Marking this one as FIXED.
              Migration made changes -
              Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
              Affects Compatibility/Configuration [ 10002 ]
              Assignee Priority P2 [ 10011 ]
              Migration made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: