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

        kasaihiroyoshi created issue -
        kenaiadmin made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 8473 40077
        Tom Mueller made changes -
        Fix Version/s not determined [ 11149 ]
        Fix Version/s V3 [ 10981 ]

          People

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

            Dates

            • Created:
              Updated: