lib: util: Add allow_access_nolog().
authorJeremy Allison <jra@samba.org>
Mon, 14 Nov 2016 19:48:32 +0000 (11:48 -0800)
committerAndreas Schneider <asn@cryptomilk.org>
Wed, 16 Nov 2016 11:41:09 +0000 (12:41 +0100)
Make allow_access() call allow_access_nolog(), then log.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12419

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
lib/util/access.c
lib/util/access.h

index 35c4920030ab46a9962482b92ecfb0a37c6e925f..859824c8a91ef1d2ef5f670055d46b1911641f96 100644 (file)
@@ -327,8 +327,8 @@ static bool allow_access_internal(const char **deny_list,
        return true;
 }
 
-/* return true if access should be allowed */
-bool allow_access(const char **deny_list,
+/* return true if access should be allowed - doesn't print log message */
+bool allow_access_nolog(const char **deny_list,
                const char **allow_list,
                const char *cname,
                const char *caddr)
@@ -339,11 +339,24 @@ bool allow_access(const char **deny_list,
 
        ret = allow_access_internal(deny_list, allow_list, nc_cname, nc_caddr);
 
+       SAFE_FREE(nc_cname);
+       SAFE_FREE(nc_caddr);
+       return ret;
+}
+
+/* return true if access should be allowed - prints log message */
+bool allow_access(const char **deny_list,
+               const char **allow_list,
+               const char *cname,
+               const char *caddr)
+{
+       bool ret;
+
+       ret = allow_access_nolog(deny_list, allow_list, cname, caddr);
+
        DEBUG(ret ? 3 : 0,
              ("%s connection from %s (%s)\n",
-              ret ? "Allowed" : "Denied", nc_cname, nc_caddr));
+              ret ? "Allowed" : "Denied", cname, caddr));
 
-       SAFE_FREE(nc_cname);
-       SAFE_FREE(nc_caddr);
        return ret;
 }
index b4dce155c6538229a24fec5ccdb29ae223dc31ca..73f71b6e4fdfdd08db686510844f0472fde452fc 100644 (file)
 bool client_match(const char *tok, const void *item);
 bool list_match(const char **list,const void *item,
                bool (*match_fn)(const char *, const void *));
+bool allow_access_nolog(const char **deny_list,
+               const char **allow_list,
+               const char *cname,
+               const char *caddr);
 bool allow_access(const char **deny_list,
                const char **allow_list,
                const char *cname,