You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by jb...@apache.org on 2013/11/12 21:14:25 UTC

svn commit: r1541216 - /karaf/trunk/shell/console/src/main/java/org/apache/karaf/shell/console/NameScoping.java

Author: jbonofre
Date: Tue Nov 12 20:14:25 2013
New Revision: 1541216

URL: http://svn.apache.org/r1541216
Log:
[KARAF-2577] Prevent NPE when no CommandSession exists

Modified:
    karaf/trunk/shell/console/src/main/java/org/apache/karaf/shell/console/NameScoping.java

Modified: karaf/trunk/shell/console/src/main/java/org/apache/karaf/shell/console/NameScoping.java
URL: http://svn.apache.org/viewvc/karaf/trunk/shell/console/src/main/java/org/apache/karaf/shell/console/NameScoping.java?rev=1541216&r1=1541215&r2=1541216&view=diff
==============================================================================
--- karaf/trunk/shell/console/src/main/java/org/apache/karaf/shell/console/NameScoping.java (original)
+++ karaf/trunk/shell/console/src/main/java/org/apache/karaf/shell/console/NameScoping.java Tue Nov 12 20:14:25 2013
@@ -50,6 +50,9 @@ public class NameScoping {
      * Returns true if the given scope is the global scope so that it can be hidden from help messages
      */
     public static boolean isGlobalScope(CommandSession session, String scope) {
+        if (session == null)
+            return false;
+
         if (!isMultiScopeMode(session)) {
             String globalScope = (String) session.get("APPLICATION");
             if (globalScope != null) {
@@ -64,6 +67,9 @@ public class NameScoping {
      * avoid prefixing commands with their scope
      */
     public static boolean isMultiScopeMode(CommandSession session) {
+        if (session == null)
+            return false;
+
         Object value = session.get(MULTI_SCOPE_MODE_KEY);
         if (value != null && value.equals("false")) {
             return false;