idmap_rfc2307: Don't stop after 30 entries

We start over again and again, so we need to search in the whole list.
This is a quick hack generating a bad O(n^2). The real fix is to
call idmap_rfc2307_find_map with "maps" starting at the right offset,
but that's an optimization for later when it's restructured

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>

diff --git a/source3/winbindd/idmap_rfc2307.c b/source3/winbindd/idmap_rfc2307.c
index deb25cc70c85632c7a7fe281ff87503d8a9d0c0b..306bc8af7629e00b2177df3b7339fc4f7975eaa4 100644 (file)
--- a/source3/winbindd/idmap_rfc2307.c
+++ b/source3/winbindd/idmap_rfc2307.c
@@ -520,10 +520,7 @@ static struct id_map* idmap_rfc2307_find_map(struct idmap_rfc2307_map *maps,
 
        DEBUG(10, ("Looking for name %s, type %d\n", name, type));
 
-       for (i = 0; i < IDMAP_LDAP_MAX_IDS; i++) {
-               if (maps[i].map == NULL) { /* end of the run */
-                       return NULL;
-               }
+       for (i = 0; maps[i].map != NULL; i++) {
                DEBUG(10, ("Entry %d: name %s, type %d\n",
                           i, maps[i].name, maps[i].type));
                if (type == maps[i].type && strcmp(name, maps[i].name) == 0) {