[jboss-cvs] JBoss Messaging SVN: r6210 - trunk/src/main/org/jboss/messaging/core/security/impl.

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

Author: jmesnil
Date: 2009-03-30 05:01:24 -0400 (Mon, 30 Mar 2009)
New Revision: 6210

Modified:
   trunk/src/main/org/jboss/messaging/core/security/impl/SecurityStoreImpl.java
Log:
fix NPE

* when sending a notification, user String can be null, create the SimpleString using the toSimpleString() static method to check for nullability

Modified: trunk/src/main/org/jboss/messaging/core/security/impl/SecurityStoreImpl.java
===================================================================
--- trunk/src/main/org/jboss/messaging/core/security/impl/SecurityStoreImpl.java	2009-03-30 07:04:26 UTC (rev 6209)
+++ trunk/src/main/org/jboss/messaging/core/security/impl/SecurityStoreImpl.java	2009-03-30 09:01:24 UTC (rev 6210)
@@ -172,7 +172,7 @@
 
                props.putStringProperty(ManagementHelper.HDR_ADDRESS, address);
                props.putStringProperty(ManagementHelper.HDR_CHECK_TYPE, new SimpleString(checkType.toString()));
-               props.putStringProperty(ManagementHelper.HDR_USER, new SimpleString(user));
+               props.putStringProperty(ManagementHelper.HDR_USER, SimpleString.toSimpleString(user));
 
                Notification notification = new Notification(NotificationType.SECURITY_PERMISSION_VIOLATION, props);