User: d_jencks
  Date: 01/12/05 13:24:00

  Modified:    src/main/org/jboss/system ServiceCreator.java
  Log:
  Fix to problem of bad DynamicMBean that returns null MBeanInfo being registered 
after MBeanServer creates it.
  
  Revision  Changes    Path
  1.5       +23 -10    jboss/src/main/org/jboss/system/ServiceCreator.java
  
  Index: ServiceCreator.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/system/ServiceCreator.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- ServiceCreator.java       2001/11/26 03:19:46     1.4
  +++ ServiceCreator.java       2001/12/05 21:23:59     1.5
  @@ -24,7 +24,7 @@
    * @see Service
    * 
    * @author <a href="mailto:[EMAIL PROTECTED]";>Marc Fleury</a>
  - * @version $Revision: 1.4 $
  + * @version $Revision: 1.5 $
    * 
    * <p><b>Revisions:</b>
    * <p><b>2001/08/03 marcf </b>
  @@ -82,17 +82,30 @@
         log.info("About to create the bean"+name);
                
         // Create the MBean instance
  -      ObjectInstance instance = 
  -         server.createMBean(code,
  -                            name,
  -                            loader,
  -                            constructor.params,
  -                            constructor.signature);
  -             
  -      log.info("Created the bean"+name);
  +      try 
  +      {
  +         ObjectInstance instance =  server.createMBean(code,
  +                                                       name,
  +                                                       loader,
  +                                                       constructor.params,
  +                                                       constructor.signature);
  +         log.info("Created the bean"+name);
                
  -      return instance;
  +         return instance;
        
  +      } 
  +      catch (Exception e) 
  +      {
  +         //didn't work, unregister in case the jmx agent is screwed.
  +         try 
  +         {
  +            server.unregisterMBean(name);
  +         } catch (Exception othere) 
  +         { } // end of try-catch
  +         
  +         throw e;
  +      } // end of try-catch
  +      
      } 
        
      /**
  
  
  

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to