glassfish
  1. glassfish
  2. GLASSFISH-8473

NPE in monitoring. it seems to be improperly MT-Unsafe.

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 9.1peur2
    • Fix Version/s: not determined
    • Component/s: web_services_mgmt
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      8,473
    • Status Whiteboard:
      Hide

      v3_exclude,V2.1.1exclude

      Show
      v3_exclude,V2.1.1exclude

      Description

      com.sun.enterprise.admin.wsmgmt.msg.MessageTraceFactory holds messages in 'Map
      _messages' and HashMap is asigned to it.

      private MessageTraceFactory()

      { _messages = new HashMap(); _msgBodyPattern = Pattern.compile("<env:Body>.*</env:Body>"); }

      HashMap is not synchronized map, and MessageTraceFactory is singleton and its
      methods seem to be called asynchronously.

      In our stress test, some NPEs occured at

      private void setResponseInfo(SOAPMessageContext ctx,
      com.sun.appserv.management.ext.wsmgmt.MessageTraceImpl trace) {
      //omit
      // sets the request size
      HERE trace.setResponseSize( baos.size() );

      and same problem is found at msgId2fc in
      com.sun.enterprise.admin.wsmgmt.agent.GlobalMessageListenerImpl.
      In our test, NPEs are caused at

      public void processRequest(String messageID,
      com.sun.enterprise.webservice.SOAPMessageContext context,
      TransportInfo info)

      { com.sun.enterprise.admin.wsmgmt.SOAPMessageContext smc = new com.sun.enterprise.admin.wsmgmt.SOAPMessageContext_2_0(context); FilterContext fc = (FilterContext) msgId2fc.get(messageID); *HERE* fc.setTransportInfo(info); fc.setMessageContext(smc); FilterRouter.getInstance().applyFilters(Filter.PROCESS_REQUEST,fc); }

        Activity

        Hide
        sirajg added a comment -

        webservices issue->jitu

        Show
        sirajg added a comment - webservices issue->jitu
        Hide
        jitu added a comment -

        This feature is not targeted for v3. Adding v3_exclude to Status whiteboard

        Show
        jitu added a comment - This feature is not targeted for v3. Adding v3_exclude to Status whiteboard
        Hide
        jagadesh added a comment -

        Will not be fixed for V2.1.1

        Show
        jagadesh added a comment - Will not be fixed for V2.1.1
        Hide
        jagadesh added a comment -

        Will not be fixed for V2.1.1

        Show
        jagadesh added a comment - Will not be fixed for V2.1.1
        Hide
        Tom Mueller added a comment -

        Bulk update to change fix version to "not determined" for all issues still open but with a fix version for a released version.

        Show
        Tom Mueller added a comment - Bulk update to change fix version to "not determined" for all issues still open but with a fix version for a released version.

          People

          • Assignee:
            jitu
            Reporter:
            kasaihiroyoshi
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: