s3-winbindd: Fix winbind NDR caching.

All of winbindd's core caching relies on NDR entries. Those entries can
not be stored in winbindd_cache.tdb via wcache_store_ndr() as long as
there is no SEQNUM entry present in the cache.

Guenther

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

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>

diff --git a/selftest/knownfail b/selftest/knownfail
index e077fbc85530c8b9ae9cff5e390beaf63c8957d7..19955ec8ca3bea87661e4c1f61f331b3700df27c 100644 (file)
--- a/selftest/knownfail
+++ b/selftest/knownfail
@@ -341,5 +341,3 @@
 
 # Currently winbindd_cache.tdb is not working as expected
 ^samba3.winbind_cache_sanity.WINBINDD_CACHE_VERSION.*
-^samba3.winbind_cache_sanity.SEQNUM.*
-^samba3.winbind_cache_sanity.NDR.*

diff --git a/source3/winbindd/winbindd_cache.c b/source3/winbindd/winbindd_cache.c
index eb79586a2ca6108c8af5a5440fbfff55a06f1602..1fb1a2f24caa195bd1149849a11e1622f63790e6 100644 (file)
--- a/source3/winbindd/winbindd_cache.c
+++ b/source3/winbindd/winbindd_cache.c
@@ -3196,6 +3196,13 @@ static bool init_wcache(void)
                return false;
        }
 
+       /*
+        * Create a dummy SEQNUM entry early, otherwise every call via the
+        * winbind NDR interface will fail to call wcache_store_ndr() when there
+        * is no SEQNUM present already
+        */
+       wcache_store_seqnum(lp_workgroup(), 0, 0);
+
        return true;
 }