diff options
author | Aidan Skinner <aidan@apache.org> | 2008-12-09 17:23:17 +0000 |
---|---|---|
committer | Aidan Skinner <aidan@apache.org> | 2008-12-09 17:23:17 +0000 |
commit | f2bec6c366dc14c9a24dd753adaf798503e87e2e (patch) | |
tree | 5aa67233657458c9de45cdc95c2ced4bad9ba8d9 /java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java | |
parent | ab8ee239590430caf25bcce32fe848575977b94c (diff) | |
download | qpid-python-f2bec6c366dc14c9a24dd753adaf798503e87e2e.tar.gz |
QPID-1469: Fix NullPointerException, add test for deleteUser which would expose this.
Patch from gemmellr@dcs.gla.ac.uk
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@724787 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java')
-rw-r--r-- | java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java b/java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java index a8ae03cc5d..32ec3a3bbc 100644 --- a/java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java +++ b/java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java @@ -441,7 +441,12 @@ public class AMQUserManagementMBean extends AMQManagedObject implements UserMana private String getCurrentJMXUser() { AccessControlContext acc = AccessController.getContext(); + Subject subject = Subject.getSubject(acc); + if (subject == null) + { + return "Unknown user, authentication Subject was null"; + } // Retrieve JMXPrincipal from Subject Set<JMXPrincipal> principals = subject.getPrincipals(JMXPrincipal.class); |