torture: Add tests for idmap cache

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
Bug: https://bugzilla.samba.org/show_bug.cgi?id=13813

diff --git a/source3/selftest/tests.py b/source3/selftest/tests.py
index 5d7e4969e5945f395526410626be5de4340f1a16..e8d516573ddd73f378820d4efbe826469ea9e38c 100755 (executable)
--- a/source3/selftest/tests.py
+++ b/source3/selftest/tests.py
@@ -200,6 +200,7 @@ local_tests = [
     "LOCAL-G-LOCK5",
     "LOCAL-G-LOCK6",
     "LOCAL-NAMEMAP-CACHE1",
+    "LOCAL-IDMAP-CACHE1",
     "LOCAL-hex_encode_buf",
     "LOCAL-remove_duplicate_addrs2"]
 

diff --git a/source3/torture/proto.h b/source3/torture/proto.h
index 669e077051e57a662ea601f89b7c638c535d484c..b4a2007fa773edc17fec3ff97d40278e68651e23 100644 (file)
--- a/source3/torture/proto.h
+++ b/source3/torture/proto.h
@@ -137,6 +137,7 @@ bool run_g_lock5(int dummy);
 bool run_g_lock6(int dummy);
 bool run_g_lock_ping_pong(int dummy);
 bool run_local_namemap_cache1(int dummy);
+bool run_local_idmap_cache1(int dummy);
 bool run_hidenewfiles(int dummy);
 
 #endif /* __TORTURE_H__ */

diff --git a/source3/torture/test_idmap_cache.c b/source3/torture/test_idmap_cache.c
new file mode 100644 (file)
index 0000000..b9cba3b
--- /dev/null
+++ b/source3/torture/test_idmap_cache.c
@@ -0,0 +1,122 @@
+/*
+ * Unix SMB/CIFS implementation.
+ * Test dbwrap_watch API
+ * Copyright (C) Volker Lendecke 2017
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include "includes.h"
+#include "torture/proto.h"
+#include "lib/idmap_cache.h"
+#include "librpc/gen_ndr/idmap.h"
+#include "libcli/security/dom_sid.h"
+
+bool run_local_idmap_cache1(int dummy)
+{
+       struct dom_sid sid, found_sid;
+       struct unixid xid, found_xid;
+       bool ret = false;
+       bool expired = false;
+
+       xid = (struct unixid) { .id = 1234, .type = ID_TYPE_UID };
+       dom_sid_parse("S-1-5-21-2864185242-3846410404-2398417794-1235", &sid);
+       idmap_cache_set_sid2unixid(&sid, &xid);
+
+       ret = idmap_cache_find_sid2unixid(&sid, &found_xid, &expired);
+       if (!ret) {
+               fprintf(stderr, "idmap_cache_find_sid2unixid failed\n");
+               goto done;
+       }
+       if (expired) {
+               fprintf(stderr,
+                       "idmap_cache_find_sid2unixid returned an expired "
+                       "value\n");
+               goto done;
+       }
+       if ((xid.type != found_xid.type) || (xid.id != found_xid.id)) {
+               fprintf(stderr,
+                       "idmap_cache_find_sid2unixid returned wrong "
+                       "values\n");
+               goto done;
+       }
+
+       ret = idmap_cache_find_xid2sid(&xid, &found_sid, &expired);
+       if (!ret) {
+               fprintf(stderr, "idmap_cache_find_xid2sid failed\n");
+               goto done;
+       }
+       if (expired) {
+               fprintf(stderr,
+                       "idmap_cache_find_xid2sid returned an expired "
+                       "value\n");
+               goto done;
+       }
+       if (!dom_sid_equal(&sid, &found_sid)) {
+               fprintf(stderr,
+                       "idmap_cache_find_xid2sid returned wrong sid\n");
+               goto done;
+       }
+
+       xid.type = ID_TYPE_GID;
+
+       ret = idmap_cache_find_xid2sid(&xid, &found_sid, &expired);
+       if (ret) {
+               fprintf(stderr,
+                       "idmap_cache_find_xid2sid found a GID where it "
+                       "should not\n");
+               goto done;
+       }
+
+       idmap_cache_del_sid(&sid);
+
+       xid.type = ID_TYPE_UID;
+       ret = idmap_cache_find_xid2sid(&xid, &found_sid, &expired);
+       if (ret) {
+               fprintf(stderr,
+                       "idmap_cache_find_xid2sid found a UID where it "
+                       "should not\n");
+               goto done;
+       }
+
+       /*
+        * Test that negative mappings can also be cached
+        */
+       sid = (struct dom_sid) {0};
+       xid = (struct unixid) { .id = 1234, .type = ID_TYPE_UID };
+       idmap_cache_set_sid2unixid(&sid, &xid);
+
+       ret = idmap_cache_find_xid2sid(&xid, &found_sid, &expired);
+       if (!ret) {
+               fprintf(stderr,
+                       "idmap_cache_find_xid2sid failed to find "
+                       "negative mapping\n");
+               goto done;
+       }
+       if (expired) {
+               fprintf(stderr,
+                       "idmap_cache_find_xid2sid returned an expired "
+                       "value\n");
+               goto done;
+       }
+       if (!dom_sid_equal(&sid, &found_sid)) {
+               fprintf(stderr,
+                       "idmap_cache_find_xid2sid returned wrong sid\n");
+               goto done;
+       }
+
+       ret = true;
+done:
+       return ret;
+}

diff --git a/source3/torture/torture.c b/source3/torture/torture.c
index b47f247356c7310f276390d71fbc7a91368e08fe..3df5e409c572533cdfc38e9890f382eb50dee317 100644 (file)
--- a/source3/torture/torture.c
+++ b/source3/torture/torture.c
@@ -12534,6 +12534,10 @@ static struct {
                .name  = "LOCAL-NAMEMAP-CACHE1",
                .fn    = run_local_namemap_cache1,
        },
+       {
+               .name  = "LOCAL-IDMAP-CACHE1",
+               .fn    = run_local_idmap_cache1,
+       },
        {
                .name  = "qpathinfo-bufsize",
                .fn    = run_qpathinfo_bufsize,

diff --git a/source3/wscript_build b/source3/wscript_build
index aa8fdc175674d8caafb9d8b327072fabe424024a..e0b983ae2b2d2641ea4b207591b7c141dc4ca7e6 100644 (file)
--- a/source3/wscript_build
+++ b/source3/wscript_build
@@ -1198,6 +1198,7 @@ bld.SAMBA3_BINARY('smbtorture' + bld.env.suffix3,
                         torture/wbc_async.c
                         torture/test_g_lock.c
                         torture/test_namemap_cache.c
+                        torture/test_idmap_cache.c
                         torture/test_hidenewfiles.c
                         ''',
                  deps='''