summaryrefslogtreecommitdiff
path: root/java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java
diff options
context:
space:
mode:
authorAidan Skinner <aidan@apache.org>2008-12-09 17:23:17 +0000
committerAidan Skinner <aidan@apache.org>2008-12-09 17:23:17 +0000
commitf2bec6c366dc14c9a24dd753adaf798503e87e2e (patch)
tree5aa67233657458c9de45cdc95c2ced4bad9ba8d9 /java/broker/src/main/java/org/apache/qpid/server/security/access/management/AMQUserManagementMBean.java
parentab8ee239590430caf25bcce32fe848575977b94c (diff)
downloadqpid-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.java5
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);