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;