autorid: use the db argument in the initialize traverse action.
authorMichael Adam <obnox@samba.org>
Wed, 2 Apr 2014 22:06:04 +0000 (00:06 +0200)
committerJeremy Allison <jra@samba.org>
Thu, 3 Apr 2014 06:36:55 +0000 (08:36 +0200)
By a copy and paste error, the global autorid_db was used.
This was not currently a problem in behaviour, because this
autorid_db is passed as the argument.

This change fixes the callback function for consistency.

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Apr  3 08:36:55 CEST 2014 on sn-devel-104

source3/winbindd/idmap_autorid.c

index 7e17b6634e06e5bde146374e213bbafac4f4b93c..a0262facb9058815ffb57e0a52d801f862689a7f 100644 (file)
@@ -592,12 +592,12 @@ static NTSTATUS idmap_autorid_initialize_action(struct db_context *db,
        common = (struct idmap_tdb_common_context *)dom->private_data;
        config = (struct autorid_global_config *)common->private_data;
 
-       status = idmap_autorid_init_hwms(autorid_db);
+       status = idmap_autorid_init_hwms(db);
        if (!NT_STATUS_IS_OK(status)) {
                return status;
        }
 
-       status = idmap_autorid_saveconfig(autorid_db, config);
+       status = idmap_autorid_saveconfig(db, config);
        if (!NT_STATUS_IS_OK(status)) {
                DEBUG(1, ("Failed to store configuration data!\n"));
                return status;