[jboss-cvs] JBossAS SVN: r100687 - projects/jboss-osgi/projects/bundles/jmx/trunk/src/main/java/org/jboss/osgi/jmx/internal.

[jboss-cvs-commits at lists.jboss.org]

Author: thomas.diesler at jboss.com
Date: 2010-02-08 02:44:17 -0500 (Mon, 08 Feb 2010)
New Revision: 100687

Modified:
   projects/jboss-osgi/projects/bundles/jmx/trunk/src/main/java/org/jboss/osgi/jmx/internal/ManagedFrameworkImpl.java
Log:
Prevent NPEs on refresh of all all bundles with trace logging enabled

Modified: projects/jboss-osgi/projects/bundles/jmx/trunk/src/main/java/org/jboss/osgi/jmx/internal/ManagedFrameworkImpl.java
===================================================================
--- projects/jboss-osgi/projects/bundles/jmx/trunk/src/main/java/org/jboss/osgi/jmx/internal/ManagedFrameworkImpl.java	2010-02-08 07:09:41 UTC (rev 100686)
+++ projects/jboss-osgi/projects/bundles/jmx/trunk/src/main/java/org/jboss/osgi/jmx/internal/ManagedFrameworkImpl.java	2010-02-08 07:44:17 UTC (rev 100687)
@@ -213,7 +213,7 @@
    public void refreshPackages(ObjectName[] objectNames)
    {
       if (log.isTraceEnabled())
-         log.trace("refreshPackages(" + Arrays.asList(objectNames) +")");
+         log.trace("refreshPackages(" + (objectNames != null ? Arrays.asList(objectNames) : null) +")");
       
       Bundle[] bundleArr = getBundles(objectNames);
       ServiceReference sref = getBundleContext().getServiceReference(PackageAdmin.class.getName());
@@ -224,7 +224,7 @@
    public boolean resolveBundles(ObjectName[] objectNames)
    {
       if (log.isTraceEnabled())
-         log.trace("resolveBundles(" + Arrays.asList(objectNames) +")");
+         log.trace("resolveBundles(" + (objectNames != null ? Arrays.asList(objectNames) : null) +")");
       
       Bundle[] bundleArr = getBundles(objectNames);
       ServiceReference sref = getBundleContext().getServiceReference(PackageAdmin.class.getName());