JMX - NanoMX registers component with JMX every time it is retrieved

Every time a JMX-managed component is retrieved from a NanoMXContainer, the NanoMXComponentAdapter tries to register it with JMX. MX4J 1.1.1 expects that this will happen only once.

Here is my proposed fix: Add a new ivar called "registered" to NanoMXComponentAdapter and return from the method before registering with JMX if the "registered" ivar is true. Look for // added by JAC below:

public Object getComponentInstance(
MutablePicoContainer componentRegistry) throws
PicoInitializationException,
PicoIntrospectionException,
AssignabilityRegistrationException,
NotConcreteRegistrationException
{
Object component = super.getComponentInstance(componentRegistry);
if (registered) return component; // added by JAC
try

{ ObjectName name = NanoMXContainer.asObjectName(getComponentKey()); Object mbean = NanoMXContainer.asMBean(component); mbeanServer.registerMBean(mbean, name); }

catch (MalformedObjectNameException e)

{ throw new NanoMXInitializationException(e); } catch (MBeanRegistrationException e) { throw new NanoMXInitializationException(e); }

catch (NotCompliantMBeanException e)

{ throw new NanoMXInitializationException(e); } catch (InstanceAlreadyExistsException e) { throw new NanoMXInitializationException(e); }

registered = true; // added by JAC
return component;
}