Attachment 15175 Details for Bug 13951 – patch for 4.10

[patch] patch for 4.10

bug-13951-fix-4-10.patch (text/plain), 3.20 KB, created by Douglas Bagnall on 2019-05-22 00:11:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Douglas Bagnall

Created: 2019-05-22 00:11:03 UTC

Size: 3.20 KB

patch

obsolete

>From d877cc06a4d986be08e85ccb02a39473598b0721 Mon Sep 17 00:00:00 2001
>From: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
>Date: Fri, 17 May 2019 14:42:24 +1200
>Subject: [PATCH] dsdb/paged_results: ignore successful results without
> messages
>
>So that we don't dereference result->msgs[0] when it doesn't exist.
>This can happen when the object has changed in such a way that it no
>longer matches the original search query.
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=13951
>
>Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
>---
> .../dsdb/samdb/ldb_modules/paged_results.c    |  3 ++-
> source4/dsdb/tests/python/vlv.py              | 21 +++++++++++++++++++
> 2 files changed, 23 insertions(+), 1 deletion(-)
>
>diff --git a/source4/dsdb/samdb/ldb_modules/paged_results.c b/source4/dsdb/samdb/ldb_modules/paged_results.c
>index 78ad44f6601..5cad398ab61 100644
>--- a/source4/dsdb/samdb/ldb_modules/paged_results.c
>+++ b/source4/dsdb/samdb/ldb_modules/paged_results.c
>@@ -266,7 +266,8 @@ static int paged_results(struct paged_context *ac)
> 		ret = paged_search_by_dn_guid(ac->module, ac, &result, guid,
> 					    ac->req->op.search.attrs,
> 					    ac->store->expr);
>-		if (ret == LDAP_NO_SUCH_OBJECT /* TODO or no result */) {
>+		if (ret == LDAP_NO_SUCH_OBJECT ||
>+		    (ret == LDB_SUCCESS && result->count == 0)) {
> 			/* The thing isn't there TODO, which we quietly
> 			   ignore and go on to send an extra one
> 			   instead. */
>diff --git a/source4/dsdb/tests/python/vlv.py b/source4/dsdb/tests/python/vlv.py
>index 8550a38e287..d67338143ce 100644
>--- a/source4/dsdb/tests/python/vlv.py
>+++ b/source4/dsdb/tests/python/vlv.py
>@@ -105,6 +105,7 @@ class TestsWithUserOU(samba.tests.TestCase):
>             'givenName': "abcdefghijklmnopqrstuvwxyz"[i % 26],
>             "roomNumber": "%sbc" % (n - i),
>             "carLicense": "åæ¥ç»",
>+            "facsimileTelephoneNumber": name,
>             "employeeNumber": "%s%sx" % (abs(i * (99 - i)), '\n' * (i & 255)),
>             "accountExpires": "%s" % (10 ** 9 + 1000000 * i),
>             "msTSExpireDate4": "19%02d0101010000.0Z" % (i % 100),
>@@ -1421,6 +1422,26 @@ class PagedResultsTests(TestsWithUserOU):
> 
>         self.assertEqual(results, expected_results)
> 
>+    def test_paged_change_during_search(self):
>+        prefix = "change_during_search_"
>+        num_users = 2
>+        users = [self.create_user(i, num_users, prefix=prefix)
>+                 for i in range(num_users)]
>+        expr = "(&(objectClass=user)(facsimileTelephoneNumber=%s*))" % (prefix)
>+
>+        # Get the first page, then change the searched attribute and
>+        # try for the second page.
>+        results, cookie = self.paged_search(expr, page_size=1)
>+
>+        for u in users:
>+            self.ldb.modify_ldif("dn: %s\n"
>+                                 "changetype: modify\n"
>+                                 "replace: facsimileTelephoneNumber\n"
>+                                 "facsimileTelephoneNumber: 123" % u['dn'])
>+
>+        results, _ = self.paged_search(expr, cookie=cookie,
>+                                       page_size=len(self.users))
>+
>     def assertPagedSearchRaises(self, err_num, expr, cookie, attrs=None,
>                                 extra_ctrls=None):
>         try:
>-- 
>2.17.1
>

Actions: View

Attachments on bug 13951: 15159 | 15165 | 15166 | 15173 | 15174 | 15175 | 15198 | 15226 | 15227 | 15228 | 15230 | 15231