[storm] branch master updated: [STORM-3767] Follow up: fix NullPointerException in getUnixGroups

[ethanli]

This is an automated email from the ASF dual-hosted git repository.

ethanli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/storm.git


The following commit(s) were added to refs/heads/master by this push:
     new aeccc45  [STORM-3767] Follow up: fix NullPointerException in 
getUnixGroups
aeccc45 is described below

commit aeccc45a126ebf45ecf68a2bc78e852c1c34074f
Author: Meng (Ethan) Li <ethanopensou...@gmail.com>
AuthorDate: Mon Sep 20 20:23:10 2021 -0500

    [STORM-3767] Follow up: fix NullPointerException in getUnixGroups
---
 .../jvm/org/apache/storm/security/auth/ShellBasedGroupsMapping.java  | 5 +++++
 1 file changed, 5 insertions(+)

diff --git 
a/storm-client/src/jvm/org/apache/storm/security/auth/ShellBasedGroupsMapping.java
 
b/storm-client/src/jvm/org/apache/storm/security/auth/ShellBasedGroupsMapping.java
index 31d00dc..c80e293 100644
--- 
a/storm-client/src/jvm/org/apache/storm/security/auth/ShellBasedGroupsMapping.java
+++ 
b/storm-client/src/jvm/org/apache/storm/security/auth/ShellBasedGroupsMapping.java
@@ -105,6 +105,11 @@ public class ShellBasedGroupsMapping implements
      * @throws IOException if encounter any error when running the command
      */
     private Set<String> getUnixGroups(final String user) throws IOException {
+        if (user == null) {
+            LOG.debug("User is null. Returning an empty set as the result");
+            return new HashSet<>();
+        }
+
         String result;
         try {
             result = 
shellCommandRunner.execCommand(ShellUtils.getGroupsForUserCommand(user));