ext_unix_group_acl: remove useless check for NULL

* the string in p is checked for NULL prior to being assigned to user
  there is no need to check it twice and doing so confuses Coverity.

 Detected by Coverity Scan. Issue 740451.

diff --git a/helpers/external_acl/unix_group/check_group.cc b/helpers/external_acl/unix_group/check_group.cc
index e0cd04b64e9ed4bce9501adb533d6e599cfd0f97..769750cce2b43b104cea85d08a519233309be53c 100644 (file)
--- a/helpers/external_acl/unix_group/check_group.cc
+++ b/helpers/external_acl/unix_group/check_group.cc
@@ -206,7 +206,7 @@ main(int argc, char *argv[])
         } else {
             user = p;
             rfc1738_unescape(user);
-            if (user && strip_dm) {
+            if (strip_dm) {
                 suser = strchr(user, '\\');
                 if (!suser) suser = strchr(user, '/');
                 if (suser && suser[1]) user = suser + 1;