#undef DBGC_CLASS
#define DBGC_CLASS DBGC_IDMAP
+/* idmap version determines auto-conversion - this is the database
+ structure version specifier. */
+
+#define IDMAP_VERSION 2
+
/* High water mark keys */
#define HWM_GROUP "GROUP HWM"
#define HWM_USER "USER HWM"
} idmap_tdb_state;
+struct convert_fn_state {
+ struct db_context *db;
+ bool failed;
+};
+
/*****************************************************************************
For idmap conversion: convert one record to new format
Ancient versions (eg 2.2.3a) of winbindd_idmap.tdb mapped DOMAINNAME/rid
instead of the SID.
*****************************************************************************/
-static int convert_fn(TDB_CONTEXT *tdb, TDB_DATA key, TDB_DATA data, void *state)
+static int convert_fn(struct db_record *rec, void *private_data)
{
struct winbindd_domain *domain;
char *p;
+ NTSTATUS status;
DOM_SID sid;
uint32 rid;
fstring keystr;
fstring dom_name;
TDB_DATA key2;
- bool *failed = (bool *)state;
+ struct convert_fn_state *s = (struct convert_fn_state *)private_data;
- DEBUG(10,("Converting %s\n", (const char *)key.dptr));
+ DEBUG(10,("Converting %s\n", (const char *)rec->key.dptr));
- p = strchr((const char *)key.dptr, '/');
+ p = strchr((const char *)rec->key.dptr, '/');
if (!p)
return 0;
*p = 0;
- fstrcpy(dom_name, (const char *)key.dptr);
+ fstrcpy(dom_name, (const char *)rec->key.dptr);
*p++ = '/';
domain = find_domain_from_name(dom_name);
if (domain == NULL) {
/* We must delete the old record. */
DEBUG(0,("Unable to find domain %s\n", dom_name ));
- DEBUG(0,("deleting record %s\n", (const char *)key.dptr ));
-
- if (tdb_delete(tdb, key) != 0) {
- DEBUG(0, ("Unable to delete record %s\n", (const char *)key.dptr));
- *failed = True;
+ DEBUG(0,("deleting record %s\n", (const char *)rec->key.dptr ));
+
+ status = rec->delete_rec(rec);
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(0, ("Unable to delete record %s:%s\n",
+ (const char *)rec->key.dptr,
+ nt_errstr(status)));
+ s->failed = true;
return -1;
}
sid_to_fstring(keystr, &sid);
key2 = string_term_tdb_data(keystr);
- if (tdb_store(tdb, key2, data, TDB_INSERT) != 0) {
- DEBUG(0,("Unable to add record %s\n", (const char *)key2.dptr ));
- *failed = True;
+ status = dbwrap_store(s->db, key2, rec->value, TDB_INSERT);
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(0,("Unable to add record %s:%s\n",
+ (const char *)key2.dptr,
+ nt_errstr(status)));
+ s->failed = true;
return -1;
}
- if (tdb_store(tdb, data, key2, TDB_REPLACE) != 0) {
- DEBUG(0,("Unable to update record %s\n", (const char *)data.dptr ));
- *failed = True;
+ status = dbwrap_store(s->db, rec->value, key2, TDB_REPLACE);
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(0,("Unable to update record %s:%s\n",
+ (const char *)rec->value.dptr,
+ nt_errstr(status)));
+ s->failed = true;
return -1;
}
- if (tdb_delete(tdb, key) != 0) {
- DEBUG(0,("Unable to delete record %s\n", (const char *)key.dptr ));
- *failed = True;
+ status = rec->delete_rec(rec);
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(0,("Unable to delete record %s:%s\n",
+ (const char *)rec->key.dptr,
+ nt_errstr(status)));
+ s->failed = true;
return -1;
}
Convert the idmap database from an older version.
*****************************************************************************/
-static bool idmap_tdb_upgrade(const char *idmap_name)
+static bool idmap_tdb_upgrade(struct db_context *db)
{
int32 vers;
bool bigendianheader;
- bool failed = False;
- TDB_CONTEXT *idmap_tdb;
+ struct convert_fn_state s;
DEBUG(0, ("Upgrading winbindd_idmap.tdb from an old version\n"));
- if (!(idmap_tdb = tdb_open_log(idmap_name, 0,
- TDB_DEFAULT, O_RDWR,
- 0600))) {
- DEBUG(0, ("Unable to open idmap database\n"));
- return False;
- }
-
- bigendianheader = (tdb_get_flags(idmap_tdb) & TDB_BIGENDIAN) ? True : False;
+ bigendianheader = (db->get_flags(db) & TDB_BIGENDIAN) ? True : False;
- vers = tdb_fetch_int32(idmap_tdb, "IDMAP_VERSION");
+ vers = dbwrap_fetch_int32(db, "IDMAP_VERSION");
if (((vers == -1) && bigendianheader) || (IREV(vers) == IDMAP_VERSION)) {
/* Arrggghh ! Bytereversed or old big-endian - make order independent ! */
int32 wm;
- wm = tdb_fetch_int32(idmap_tdb, HWM_USER);
+ wm = dbwrap_fetch_int32(db, HWM_USER);
if (wm != -1) {
wm = IREV(wm);
wm = idmap_tdb_state.low_uid;
}
- if (tdb_store_int32(idmap_tdb, HWM_USER, wm) == -1) {
+ if (dbwrap_store_int32(db, HWM_USER, wm) == -1) {
DEBUG(0, ("Unable to byteswap user hwm in idmap database\n"));
- tdb_close(idmap_tdb);
return False;
}
- wm = tdb_fetch_int32(idmap_tdb, HWM_GROUP);
+ wm = dbwrap_fetch_int32(db, HWM_GROUP);
if (wm != -1) {
wm = IREV(wm);
} else {
wm = idmap_tdb_state.low_gid;
}
- if (tdb_store_int32(idmap_tdb, HWM_GROUP, wm) == -1) {
+ if (dbwrap_store_int32(db, HWM_GROUP, wm) == -1) {
DEBUG(0, ("Unable to byteswap group hwm in idmap database\n"));
- tdb_close(idmap_tdb);
return False;
}
}
+ s.db = db;
+ s.failed = false;
+
/* the old format stored as DOMAIN/rid - now we store the SID direct */
- tdb_traverse(idmap_tdb, convert_fn, &failed);
+ db->traverse(db, convert_fn, &s);
- if (failed) {
+ if (s.failed) {
DEBUG(0, ("Problem during conversion\n"));
- tdb_close(idmap_tdb);
return False;
}
- if (tdb_store_int32(idmap_tdb, "IDMAP_VERSION", IDMAP_VERSION) == -1) {
+ if (dbwrap_store_int32(db, "IDMAP_VERSION", IDMAP_VERSION) == -1) {
DEBUG(0, ("Unable to dtore idmap version in databse\n"));
- tdb_close(idmap_tdb);
return False;
}
- tdb_close(idmap_tdb);
return True;
}
-/* WARNING: We can't open a tdb twice inthe same process, for that reason
- * I'm going to use a hack with open ref counts to open the winbindd_idmap.tdb
- * only once. We will later decide whether to split the db in multiple files
- * or come up with a better solution to share them. */
+static NTSTATUS idmap_tdb_load_ranges(void)
+{
+ uid_t low_uid = 0;
+ uid_t high_uid = 0;
+ gid_t low_gid = 0;
+ gid_t high_gid = 0;
+
+ if (!lp_idmap_uid(&low_uid, &high_uid)) {
+ DEBUG(1, ("idmap uid missing\n"));
+ return NT_STATUS_UNSUCCESSFUL;
+ }
+
+ if (!lp_idmap_gid(&low_gid, &high_gid)) {
+ DEBUG(1, ("idmap gid missing\n"));
+ return NT_STATUS_UNSUCCESSFUL;
+ }
+
+ idmap_tdb_state.low_uid = low_uid;
+ idmap_tdb_state.high_uid = high_uid;
+ idmap_tdb_state.low_gid = low_gid;
+ idmap_tdb_state.high_gid = high_gid;
-static TDB_CONTEXT *idmap_tdb_common_ctx;
-static int idmap_tdb_open_ref_count = 0;
+ if (idmap_tdb_state.high_uid <= idmap_tdb_state.low_uid) {
+ DEBUG(1, ("idmap uid range missing or invalid\n"));
+ return NT_STATUS_UNSUCCESSFUL;
+ }
+
+ if (idmap_tdb_state.high_gid <= idmap_tdb_state.low_gid) {
+ DEBUG(1, ("idmap gid range missing or invalid\n"));
+ return NT_STATUS_UNSUCCESSFUL;
+ }
+
+ return NT_STATUS_OK;
+}
-static NTSTATUS idmap_tdb_open_db(TALLOC_CTX *memctx, TDB_CONTEXT **tdbctx)
+static NTSTATUS idmap_tdb_open_db(TALLOC_CTX *memctx,
+ bool check_config,
+ struct db_context **dbctx)
{
NTSTATUS ret;
TALLOC_CTX *ctx;
- SMB_STRUCT_STAT stbuf;
char *tdbfile = NULL;
- int32 version;
- bool tdb_is_new = False;
-
- if (idmap_tdb_open_ref_count) { /* the tdb has already been opened */
- idmap_tdb_open_ref_count++;
- *tdbctx = idmap_tdb_common_ctx;
- return NT_STATUS_OK;
+ struct db_context *db = NULL;
+ int32_t version;
+ bool config_error = false;
+
+ ret = idmap_tdb_load_ranges();
+ if (!NT_STATUS_IS_OK(ret)) {
+ config_error = true;
+ if (check_config) {
+ return ret;
+ }
}
/* use our own context here */
- ctx = talloc_new(memctx);
- if (!ctx) {
- DEBUG(0, ("Out of memory!\n"));
- return NT_STATUS_NO_MEMORY;
- }
+ ctx = talloc_stackframe();
/* use the old database if present */
- tdbfile = talloc_strdup(ctx, state_path("winbindd_idmap.tdb"));
+ tdbfile = state_path("winbindd_idmap.tdb");
if (!tdbfile) {
DEBUG(0, ("Out of memory!\n"));
ret = NT_STATUS_NO_MEMORY;
goto done;
}
- if (!file_exist(tdbfile, &stbuf)) {
- tdb_is_new = True;
- }
-
DEBUG(10,("Opening tdbfile %s\n", tdbfile ));
/* Open idmap repository */
- if (!(idmap_tdb_common_ctx = tdb_open_log(tdbfile, 0, TDB_DEFAULT, O_RDWR | O_CREAT, 0644))) {
+ db = db_open(ctx, tdbfile, 0, TDB_DEFAULT, O_RDWR | O_CREAT, 0644);
+ if (!db) {
DEBUG(0, ("Unable to open idmap database\n"));
ret = NT_STATUS_UNSUCCESSFUL;
goto done;
}
- if (tdb_is_new) {
- /* the file didn't existed before opening it, let's
- * store idmap version as nobody else yet opened and
- * stored it. I do not like this method but didn't
- * found a way to understand if an opened tdb have
- * been just created or not --- SSS */
- tdb_store_int32(idmap_tdb_common_ctx, "IDMAP_VERSION", IDMAP_VERSION);
- }
-
/* check against earlier versions */
- version = tdb_fetch_int32(idmap_tdb_common_ctx, "IDMAP_VERSION");
+ version = dbwrap_fetch_int32(db, "IDMAP_VERSION");
if (version != IDMAP_VERSION) {
-
- /* backup_tdb expects the tdb not to be open */
- tdb_close(idmap_tdb_common_ctx);
+ if (config_error) {
+ DEBUG(0,("Upgrade of IDMAP_VERSION from %d to %d is not "
+ "possible with incomplete configuration\n",
+ version, IDMAP_VERSION));
+ ret = NT_STATUS_UNSUCCESSFUL;
+ goto done;
+ }
+ if (db->transaction_start(db) != 0) {
+ DEBUG(0, ("Unable to start upgrade transaction!\n"));
+ ret = NT_STATUS_INTERNAL_DB_ERROR;
+ goto done;
+ }
- if ( ! idmap_tdb_upgrade(tdbfile)) {
-
+ if (!idmap_tdb_upgrade(db)) {
+ db->transaction_cancel(db);
DEBUG(0, ("Unable to open idmap database, it's in an old formati, and upgrade failed!\n"));
ret = NT_STATUS_INTERNAL_DB_ERROR;
goto done;
}
- /* Re-Open idmap repository */
- if (!(idmap_tdb_common_ctx = tdb_open_log(tdbfile, 0, TDB_DEFAULT, O_RDWR | O_CREAT, 0644))) {
- DEBUG(0, ("Unable to open idmap database\n"));
- ret = NT_STATUS_UNSUCCESSFUL;
+ if (db->transaction_commit(db) != 0) {
+ DEBUG(0, ("Unable to commit upgrade transaction!\n"));
+ ret = NT_STATUS_INTERNAL_DB_ERROR;
goto done;
}
}
- *tdbctx = idmap_tdb_common_ctx;
- idmap_tdb_open_ref_count++;
+ *dbctx = talloc_move(memctx, &db);
ret = NT_STATUS_OK;
done:
return ret;
}
- /* NEVER use tdb_close() except for the conversion routines that are guaranteed
- * to run only when the database is opened the first time, always use this function. */
-
-bool idmap_tdb_tdb_close(TDB_CONTEXT *tdbctx)
-{
- if (tdbctx != idmap_tdb_common_ctx) {
- DEBUG(0, ("ERROR: Invalid tdb context!"));
- return False;
- }
-
- idmap_tdb_open_ref_count--;
- if (idmap_tdb_open_ref_count) {
- return True;
- }
-
- return tdb_close(idmap_tdb_common_ctx);
-}
-
/**********************************************************************
IDMAP ALLOC TDB BACKEND
**********************************************************************/
-static TDB_CONTEXT *idmap_alloc_tdb;
+static struct db_context *idmap_alloc_db;
/**********************************
Initialise idmap alloc database.
static NTSTATUS idmap_tdb_alloc_init( const char *params )
{
- NTSTATUS ret;
- TALLOC_CTX *ctx;
- const char *range;
- uid_t low_uid = 0;
- uid_t high_uid = 0;
- gid_t low_gid = 0;
- gid_t high_gid = 0;
+ int ret;
+ NTSTATUS status;
+ uint32_t low_uid;
+ uint32_t low_gid;
+ bool update_uid = false;
+ bool update_gid = false;
- /* use our own context here */
- ctx = talloc_new(NULL);
- if (!ctx) {
- DEBUG(0, ("Out of memory!\n"));
- return NT_STATUS_NO_MEMORY;
+ status = idmap_tdb_open_db(NULL, true, &idmap_alloc_db);
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(0, ("idmap will be unable to map foreign SIDs: %s\n",
+ nt_errstr(status)));
+ return status;
}
- ret = idmap_tdb_open_db(ctx, &idmap_alloc_tdb);
- if ( ! NT_STATUS_IS_OK(ret)) {
- talloc_free(ctx);
- return ret;
+ low_uid = dbwrap_fetch_int32(idmap_alloc_db, HWM_USER);
+ if (low_uid == -1 || low_uid < idmap_tdb_state.low_uid) {
+ update_uid = true;
}
- talloc_free(ctx);
-
- /* load ranges */
- idmap_tdb_state.low_uid = 0;
- idmap_tdb_state.high_uid = 0;
- idmap_tdb_state.low_gid = 0;
- idmap_tdb_state.high_gid = 0;
-
- range = lp_parm_const_string(-1, "idmap alloc config", "range", NULL);
- if (range && range[0]) {
- unsigned low_id, high_id;
-
- if (sscanf(range, "%u - %u", &low_id, &high_id) == 2) {
- if (low_id < high_id) {
- idmap_tdb_state.low_gid = idmap_tdb_state.low_uid = low_id;
- idmap_tdb_state.high_gid = idmap_tdb_state.high_uid = high_id;
- } else {
- DEBUG(1, ("ERROR: invalid idmap alloc range [%s]", range));
- }
- } else {
- DEBUG(1, ("ERROR: invalid syntax for idmap alloc config:range [%s]", range));
- }
+ low_gid = dbwrap_fetch_int32(idmap_alloc_db, HWM_GROUP);
+ if (low_gid == -1 || low_gid < idmap_tdb_state.low_gid) {
+ update_gid = true;
}
- /* Create high water marks for group and user id */
- if (lp_idmap_uid(&low_uid, &high_uid)) {
- idmap_tdb_state.low_uid = low_uid;
- idmap_tdb_state.high_uid = high_uid;
+ if (!update_uid && !update_gid) {
+ return NT_STATUS_OK;
}
- if (lp_idmap_gid(&low_gid, &high_gid)) {
- idmap_tdb_state.low_gid = low_gid;
- idmap_tdb_state.high_gid = high_gid;
+ if (idmap_alloc_db->transaction_start(idmap_alloc_db) != 0) {
+ TALLOC_FREE(idmap_alloc_db);
+ DEBUG(0, ("Unable to start upgrade transaction!\n"));
+ return NT_STATUS_INTERNAL_DB_ERROR;
}
- if (idmap_tdb_state.high_uid <= idmap_tdb_state.low_uid) {
- DEBUG(1, ("idmap uid range missing or invalid\n"));
- DEBUGADD(1, ("idmap will be unable to map foreign SIDs\n"));
- return NT_STATUS_UNSUCCESSFUL;
- } else {
- uint32 low_id;
-
- if (((low_id = tdb_fetch_int32(idmap_alloc_tdb, HWM_USER)) == -1) ||
- (low_id < idmap_tdb_state.low_uid)) {
- if (tdb_store_int32(idmap_alloc_tdb, HWM_USER, idmap_tdb_state.low_uid) == -1) {
- DEBUG(0, ("Unable to initialise user hwm in idmap database\n"));
- return NT_STATUS_INTERNAL_DB_ERROR;
- }
+ if (update_uid) {
+ ret = dbwrap_store_int32(idmap_alloc_db, HWM_USER,
+ idmap_tdb_state.low_uid);
+ if (ret == -1) {
+ idmap_alloc_db->transaction_cancel(idmap_alloc_db);
+ TALLOC_FREE(idmap_alloc_db);
+ DEBUG(0, ("Unable to initialise user hwm in idmap "
+ "database\n"));
+ return NT_STATUS_INTERNAL_DB_ERROR;
}
}
- if (idmap_tdb_state.high_gid <= idmap_tdb_state.low_gid) {
- DEBUG(1, ("idmap gid range missing or invalid\n"));
- DEBUGADD(1, ("idmap will be unable to map foreign SIDs\n"));
- return NT_STATUS_UNSUCCESSFUL;
- } else {
- uint32 low_id;
-
- if (((low_id = tdb_fetch_int32(idmap_alloc_tdb, HWM_GROUP)) == -1) ||
- (low_id < idmap_tdb_state.low_gid)) {
- if (tdb_store_int32(idmap_alloc_tdb, HWM_GROUP, idmap_tdb_state.low_gid) == -1) {
- DEBUG(0, ("Unable to initialise group hwm in idmap database\n"));
- return NT_STATUS_INTERNAL_DB_ERROR;
- }
+ if (update_gid) {
+ ret = dbwrap_store_int32(idmap_alloc_db, HWM_GROUP,
+ idmap_tdb_state.low_gid);
+ if (ret == -1) {
+ idmap_alloc_db->transaction_cancel(idmap_alloc_db);
+ TALLOC_FREE(idmap_alloc_db);
+ DEBUG(0, ("Unable to initialise group hwm in idmap "
+ "database\n"));
+ return NT_STATUS_INTERNAL_DB_ERROR;
}
}
+ if (idmap_alloc_db->transaction_commit(idmap_alloc_db) != 0) {
+ TALLOC_FREE(idmap_alloc_db);
+ DEBUG(0, ("Unable to commit upgrade transaction!\n"));
+ return NT_STATUS_INTERNAL_DB_ERROR;
+ }
+
return NT_STATUS_OK;
}
const char *hwmtype;
uint32_t high_hwm;
uint32_t hwm;
+ int res;
/* Get current high water mark */
switch (xid->type) {
return NT_STATUS_INVALID_PARAMETER;
}
- if ((hwm = tdb_fetch_int32(idmap_alloc_tdb, hwmkey)) == -1) {
+ res = idmap_alloc_db->transaction_start(idmap_alloc_db);
+ if (res != 0) {
+ DEBUG(1, (__location__ " Failed to start transaction.\n"));
+ return NT_STATUS_UNSUCCESSFUL;
+ }
+
+ if ((hwm = dbwrap_fetch_int32(idmap_alloc_db, hwmkey)) == -1) {
+ idmap_alloc_db->transaction_cancel(idmap_alloc_db);
return NT_STATUS_INTERNAL_DB_ERROR;
}
if (hwm > high_hwm) {
DEBUG(1, ("Fatal Error: %s range full!! (max: %lu)\n",
hwmtype, (unsigned long)high_hwm));
+ idmap_alloc_db->transaction_cancel(idmap_alloc_db);
return NT_STATUS_UNSUCCESSFUL;
}
/* fetch a new id and increment it */
- ret = tdb_change_uint32_atomic(idmap_alloc_tdb, hwmkey, &hwm, 1);
- if (!ret) {
- DEBUG(1, ("Fatal error while fetching a new %s value\n!", hwmtype));
+ ret = dbwrap_change_uint32_atomic(idmap_alloc_db, hwmkey, &hwm, 1);
+ if (ret != 0) {
+ DEBUG(0, ("Fatal error while fetching a new %s value\n!", hwmtype));
+ idmap_alloc_db->transaction_cancel(idmap_alloc_db);
return NT_STATUS_UNSUCCESSFUL;
}
if (hwm > high_hwm) {
DEBUG(1, ("Fatal Error: %s range full!! (max: %lu)\n",
hwmtype, (unsigned long)high_hwm));
+ idmap_alloc_db->transaction_cancel(idmap_alloc_db);
return NT_STATUS_UNSUCCESSFUL;
}
-
+
+ res = idmap_alloc_db->transaction_commit(idmap_alloc_db);
+ if (res != 0) {
+ DEBUG(1, (__location__ " Failed to commit transaction.\n"));
+ return NT_STATUS_UNSUCCESSFUL;
+ }
+
xid->id = hwm;
DEBUG(10,("New %s = %d\n", hwmtype, hwm));
return NT_STATUS_INVALID_PARAMETER;
}
- if ((hwm = tdb_fetch_int32(idmap_alloc_tdb, hwmkey)) == -1) {
+ if ((hwm = dbwrap_fetch_int32(idmap_alloc_db, hwmkey)) == -1) {
return NT_STATUS_INTERNAL_DB_ERROR;
}
hwm = xid->id;
- if ((hwm = tdb_store_int32(idmap_alloc_tdb, hwmkey, hwm)) == -1) {
+ if ((hwm = dbwrap_store_uint32(idmap_alloc_db, hwmkey, hwm)) == -1) {
return NT_STATUS_INTERNAL_DB_ERROR;
}
static NTSTATUS idmap_tdb_alloc_close(void)
{
- if (idmap_alloc_tdb) {
- if (idmap_tdb_tdb_close(idmap_alloc_tdb) == 0) {
- return NT_STATUS_OK;
- } else {
- return NT_STATUS_UNSUCCESSFUL;
- }
- }
+ TALLOC_FREE(idmap_alloc_db);
return NT_STATUS_OK;
}
**********************************************************************/
struct idmap_tdb_context {
- TDB_CONTEXT *tdb;
+ struct db_context *db;
uint32_t filter_low_id;
uint32_t filter_high_id;
};
Initialise idmap database.
*****************************/
-static NTSTATUS idmap_tdb_db_init(struct idmap_domain *dom)
+static NTSTATUS idmap_tdb_db_init(struct idmap_domain *dom, const char *params)
{
NTSTATUS ret;
struct idmap_tdb_context *ctx;
goto failed;
}
- ret = idmap_tdb_open_db(ctx, &ctx->tdb);
+ ret = idmap_tdb_open_db(ctx, false, &ctx->db);
if ( ! NT_STATUS_IS_OK(ret)) {
goto failed;
}
}
dom->private_data = ctx;
- dom->initialized = True;
talloc_free(config_option);
return NT_STATUS_OK;
DEBUG(10,("Fetching record %s\n", keystr));
/* Check if the mapping exists */
- data = tdb_fetch_bystring(ctx->tdb, keystr);
+ data = dbwrap_fetch_bystring(ctx->db, NULL, keystr);
if (!data.dptr) {
DEBUG(10,("Record %s not found\n", keystr));
ret = NT_STATUS_OK;
done:
- SAFE_FREE(data.dptr);
+ talloc_free(data.dptr);
talloc_free(keystr);
return ret;
}
TDB_DATA data;
char *keystr;
unsigned long rec_id = 0;
- fstring tmp;
+ TALLOC_CTX *tmp_ctx = talloc_stackframe();
- if ((keystr = talloc_asprintf(
- ctx, "%s", sid_to_fstring(tmp, map->sid))) == NULL) {
+ keystr = sid_string_talloc(tmp_ctx, map->sid);
+ if (keystr == NULL) {
DEBUG(0, ("Out of memory!\n"));
ret = NT_STATUS_NO_MEMORY;
goto done;
DEBUG(10,("Fetching record %s\n", keystr));
/* Check if sid is present in database */
- data = tdb_fetch_bystring(ctx->tdb, keystr);
+ data = dbwrap_fetch_bystring(ctx->db, tmp_ctx, keystr);
if (!data.dptr) {
DEBUG(10,("Record %s not found\n", keystr));
ret = NT_STATUS_NONE_MAPPED;
DEBUG(2, ("Found INVALID record %s -> %s\n", keystr, (const char *)data.dptr));
ret = NT_STATUS_INTERNAL_DB_ERROR;
}
-
- SAFE_FREE(data.dptr);
/* apply filters before returning result */
if ((ctx->filter_low_id && (map->xid.id < ctx->filter_low_id)) ||
}
done:
- talloc_free(keystr);
+ talloc_free(tmp_ctx);
return ret;
}
NTSTATUS ret;
int i;
- /* make sure we initialized */
- if ( ! dom->initialized) {
- ret = idmap_tdb_db_init(dom);
- if ( ! NT_STATUS_IS_OK(ret)) {
- return ret;
- }
+ /* initialize the status to avoid suprise */
+ for (i = 0; ids[i]; i++) {
+ ids[i]->status = ID_UNKNOWN;
}
-
+
ctx = talloc_get_type(dom->private_data, struct idmap_tdb_context);
for (i = 0; ids[i]; i++) {
NTSTATUS ret;
int i;
- /* make sure we initialized */
- if ( ! dom->initialized) {
- ret = idmap_tdb_db_init(dom);
- if ( ! NT_STATUS_IS_OK(ret)) {
- return ret;
- }
+ /* initialize the status to avoid suprise */
+ for (i = 0; ids[i]; i++) {
+ ids[i]->status = ID_UNKNOWN;
}
-
+
ctx = talloc_get_type(dom->private_data, struct idmap_tdb_context);
for (i = 0; ids[i]; i++) {
}
/**********************************
- set a mapping.
+ set a mapping.
**********************************/
-static NTSTATUS idmap_tdb_set_mapping(struct idmap_domain *dom, const struct id_map *map)
+static NTSTATUS idmap_tdb_set_mapping(struct idmap_domain *dom,
+ const struct id_map *map)
{
struct idmap_tdb_context *ctx;
NTSTATUS ret;
- TDB_DATA ksid, kid, data;
+ TDB_DATA ksid, kid;
char *ksidstr, *kidstr;
fstring tmp;
- /* make sure we initialized */
- if ( ! dom->initialized) {
- ret = idmap_tdb_db_init(dom);
- if ( ! NT_STATUS_IS_OK(ret)) {
- return ret;
- }
- }
-
if (!map || !map->sid) {
return NT_STATUS_INVALID_PARAMETER;
}
ksidstr = kidstr = NULL;
- data.dptr = NULL;
/* TODO: should we filter a set_mapping using low/high filters ? */
-
+
ctx = talloc_get_type(dom->private_data, struct idmap_tdb_context);
switch (map->xid.type) {
case ID_TYPE_UID:
- kidstr = talloc_asprintf(ctx, "UID %lu", (unsigned long)map->xid.id);
+ kidstr = talloc_asprintf(ctx, "UID %lu",
+ (unsigned long)map->xid.id);
break;
-
+
case ID_TYPE_GID:
- kidstr = talloc_asprintf(ctx, "GID %lu", (unsigned long)map->xid.id);
+ kidstr = talloc_asprintf(ctx, "GID %lu",
+ (unsigned long)map->xid.id);
break;
default:
kid = string_term_tdb_data(kidstr);
ksid = string_term_tdb_data(ksidstr);
- /* *DELETE* previous mappings if any.
- * This is done both SID and [U|G]ID passed in */
-
- /* Lock the record for this SID. */
- if (tdb_chainlock(ctx->tdb, ksid) != 0) {
- DEBUG(10,("Failed to lock record %s. Error %s\n",
- ksidstr, tdb_errorstr(ctx->tdb) ));
- return NT_STATUS_UNSUCCESSFUL;
- }
-
- data = tdb_fetch(ctx->tdb, ksid);
- if (data.dptr) {
- DEBUG(10, ("Deleting existing mapping %s <-> %s\n", (const char *)data.dptr, ksidstr ));
- tdb_delete(ctx->tdb, data);
- tdb_delete(ctx->tdb, ksid);
- SAFE_FREE(data.dptr);
+ if (ctx->db->transaction_start(ctx->db) != 0) {
+ DEBUG(0, ("Failed to start transaction for %s\n",
+ ksidstr));
+ ret = NT_STATUS_INTERNAL_DB_ERROR;
+ goto done;
}
- data = tdb_fetch(ctx->tdb, kid);
- if (data.dptr) {
- DEBUG(10,("Deleting existing mapping %s <-> %s\n", (const char *)data.dptr, kidstr ));
- tdb_delete(ctx->tdb, data);
- tdb_delete(ctx->tdb, kid);
- SAFE_FREE(data.dptr);
+ ret = dbwrap_store(ctx->db, ksid, kid, TDB_REPLACE);
+ if (!NT_STATUS_IS_OK(ret)) {
+ ctx->db->transaction_cancel(ctx->db);
+ DEBUG(0, ("Error storing SID -> ID (%s -> %s): %s\n",
+ ksidstr, kidstr, nt_errstr(ret)));
+ goto done;
}
-
- if (tdb_store(ctx->tdb, ksid, kid, TDB_INSERT) == -1) {
- DEBUG(0, ("Error storing SID -> ID: %s\n", tdb_errorstr(ctx->tdb)));
- tdb_chainunlock(ctx->tdb, ksid);
- ret = NT_STATUS_UNSUCCESSFUL;
+ ret = dbwrap_store(ctx->db, kid, ksid, TDB_REPLACE);
+ if (!NT_STATUS_IS_OK(ret)) {
+ ctx->db->transaction_cancel(ctx->db);
+ DEBUG(0, ("Error storing ID -> SID (%s -> %s): %s\n",
+ kidstr, ksidstr, nt_errstr(ret)));
goto done;
}
- if (tdb_store(ctx->tdb, kid, ksid, TDB_INSERT) == -1) {
- DEBUG(0, ("Error stroing ID -> SID: %s\n", tdb_errorstr(ctx->tdb)));
- /* try to remove the previous stored SID -> ID map */
- tdb_delete(ctx->tdb, ksid);
- tdb_chainunlock(ctx->tdb, ksid);
- ret = NT_STATUS_UNSUCCESSFUL;
+
+ if (ctx->db->transaction_commit(ctx->db) != 0) {
+ DEBUG(0, ("Failed to commit transaction for (%s -> %s)\n",
+ ksidstr, kidstr));
+ ret = NT_STATUS_INTERNAL_DB_ERROR;
goto done;
}
- tdb_chainunlock(ctx->tdb, ksid);
DEBUG(10,("Stored %s <-> %s\n", ksidstr, kidstr));
ret = NT_STATUS_OK;
done:
talloc_free(ksidstr);
talloc_free(kidstr);
- SAFE_FREE(data.dptr);
return ret;
}
/**********************************
- remove a mapping.
+ remove a mapping.
**********************************/
-static NTSTATUS idmap_tdb_remove_mapping(struct idmap_domain *dom, const struct id_map *map)
+static NTSTATUS idmap_tdb_remove_mapping(struct idmap_domain *dom,
+ const struct id_map *map)
{
struct idmap_tdb_context *ctx;
NTSTATUS ret;
char *ksidstr, *kidstr;
fstring tmp;
- /* make sure we initialized */
- if ( ! dom->initialized) {
- ret = idmap_tdb_db_init(dom);
- if ( ! NT_STATUS_IS_OK(ret)) {
- return ret;
- }
- }
-
if (!map || !map->sid) {
return NT_STATUS_INVALID_PARAMETER;
}
data.dptr = NULL;
/* TODO: should we filter a remove_mapping using low/high filters ? */
-
+
ctx = talloc_get_type(dom->private_data, struct idmap_tdb_context);
switch (map->xid.type) {
case ID_TYPE_UID:
- kidstr = talloc_asprintf(ctx, "UID %lu", (unsigned long)map->xid.id);
+ kidstr = talloc_asprintf(ctx, "UID %lu",
+ (unsigned long)map->xid.id);
break;
-
+
case ID_TYPE_GID:
- kidstr = talloc_asprintf(ctx, "GID %lu", (unsigned long)map->xid.id);
+ kidstr = talloc_asprintf(ctx, "GID %lu",
+ (unsigned long)map->xid.id);
break;
default:
ksid = string_term_tdb_data(ksidstr);
kid = string_term_tdb_data(kidstr);
- /* Lock the record for this SID. */
- if (tdb_chainlock(ctx->tdb, ksid) != 0) {
- DEBUG(10,("Failed to lock record %s. Error %s\n",
- ksidstr, tdb_errorstr(ctx->tdb) ));
- return NT_STATUS_UNSUCCESSFUL;
+ if (ctx->db->transaction_start(ctx->db) != 0) {
+ DEBUG(0, ("Failed to start transaction for %s\n",
+ ksidstr));
+ return NT_STATUS_INTERNAL_DB_ERROR;
}
/* Check if sid is present in database */
- data = tdb_fetch(ctx->tdb, ksid);
+ data = dbwrap_fetch(ctx->db, NULL, ksid);
if (!data.dptr) {
+ ctx->db->transaction_cancel(ctx->db);
DEBUG(10,("Record %s not found\n", ksidstr));
- tdb_chainunlock(ctx->tdb, ksid);
ret = NT_STATUS_NONE_MAPPED;
goto done;
}
/* Check if sid is mapped to the specified ID */
if ((data.dsize != kid.dsize) ||
(memcmp(data.dptr, kid.dptr, data.dsize) != 0)) {
+ ctx->db->transaction_cancel(ctx->db);
DEBUG(10,("Specified SID does not map to specified ID\n"));
- DEBUGADD(10,("Actual mapping is %s -> %s\n", ksidstr, (const char *)data.dptr));
- tdb_chainunlock(ctx->tdb, ksid);
+ DEBUGADD(10,("Actual mapping is %s -> %s\n", ksidstr,
+ (const char *)data.dptr));
ret = NT_STATUS_NONE_MAPPED;
goto done;
}
-
+
DEBUG(10, ("Removing %s <-> %s map\n", ksidstr, kidstr));
/* Delete previous mappings. */
-
+
DEBUG(10, ("Deleting existing mapping %s -> %s\n", ksidstr, kidstr ));
- tdb_delete(ctx->tdb, ksid);
+ ret = dbwrap_delete(ctx->db, ksid);
+ if (!NT_STATUS_IS_OK(ret)) {
+ DEBUG(0,("Warning: Failed to delete %s: %s\n",
+ ksidstr, nt_errstr(ret)));
+ }
DEBUG(10,("Deleting existing mapping %s -> %s\n", kidstr, ksidstr ));
- tdb_delete(ctx->tdb, kid);
+ ret = dbwrap_delete(ctx->db, kid);
+ if (!NT_STATUS_IS_OK(ret)) {
+ DEBUG(0,("Warning: Failed to delete %s: %s\n",
+ kidstr, nt_errstr(ret)));
+ }
+
+ if (ctx->db->transaction_commit(ctx->db) != 0) {
+ DEBUG(0, ("Failed to commit transaction for (%s -> %s)\n",
+ ksidstr, kidstr));
+ ret = NT_STATUS_INTERNAL_DB_ERROR;
+ goto done;
+ }
- tdb_chainunlock(ctx->tdb, ksid);
ret = NT_STATUS_OK;
done:
talloc_free(ksidstr);
talloc_free(kidstr);
- SAFE_FREE(data.dptr);
+ talloc_free(data.dptr);
return ret;
}
if (dom->private_data) {
ctx = talloc_get_type(dom->private_data, struct idmap_tdb_context);
- if (idmap_tdb_tdb_close(ctx->tdb) == 0) {
- return NT_STATUS_OK;
- } else {
- return NT_STATUS_UNSUCCESSFUL;
- }
+ TALLOC_FREE(ctx->db);
}
return NT_STATUS_OK;
}
NTSTATUS ret;
};
-static int idmap_tdb_dump_one_entry(TDB_CONTEXT *tdb, TDB_DATA key, TDB_DATA value, void *pdata)
+static int idmap_tdb_dump_one_entry(struct db_record *rec, void *pdata)
{
struct dump_data *data = talloc_get_type(pdata, struct dump_data);
struct id_map *maps;
int num_maps = *data->num_maps;
/* ignore any record but the ones with a SID as key */
- if (strncmp((const char *)key.dptr, "S-", 2) == 0) {
+ if (strncmp((const char *)rec->key.dptr, "S-", 2) == 0) {
maps = talloc_realloc(NULL, *data->maps, struct id_map, num_maps+1);
if ( ! maps) {
return -1;
}
- if (!string_to_sid(maps[num_maps].sid, (const char *)key.dptr)) {
- DEBUG(10,("INVALID record %s\n", (const char *)key.dptr));
+ if (!string_to_sid(maps[num_maps].sid, (const char *)rec->key.dptr)) {
+ DEBUG(10,("INVALID record %s\n", (const char *)rec->key.dptr));
/* continue even with errors */
return 0;
}
/* Try a UID record. */
- if (sscanf((const char *)value.dptr, "UID %u", &(maps[num_maps].xid.id)) == 1) {
+ if (sscanf((const char *)rec->value.dptr, "UID %u", &(maps[num_maps].xid.id)) == 1) {
maps[num_maps].xid.type = ID_TYPE_UID;
maps[num_maps].status = ID_MAPPED;
*data->num_maps = num_maps + 1;
/* Try a GID record. */
} else
- if (sscanf((const char *)value.dptr, "GID %u", &(maps[num_maps].xid.id)) == 1) {
+ if (sscanf((const char *)rec->value.dptr, "GID %u", &(maps[num_maps].xid.id)) == 1) {
maps[num_maps].xid.type = ID_TYPE_GID;
maps[num_maps].status = ID_MAPPED;
*data->num_maps = num_maps + 1;
} else {
maps[num_maps].status = ID_UNKNOWN;
DEBUG(2, ("Found INVALID record %s -> %s\n",
- (const char *)key.dptr, (const char *)value.dptr));
+ (const char *)rec->key.dptr,
+ (const char *)rec->value.dptr));
/* do not increment num_maps */
}
}
struct dump_data *data;
NTSTATUS ret = NT_STATUS_OK;
- /* make sure we initialized */
- if ( ! dom->initialized) {
- ret = idmap_tdb_db_init(dom);
- if ( ! NT_STATUS_IS_OK(ret)) {
- return ret;
- }
- }
-
ctx = talloc_get_type(dom->private_data, struct idmap_tdb_context);
data = TALLOC_ZERO_P(ctx, struct dump_data);
data->num_maps = num_maps;
data->ret = NT_STATUS_OK;
- tdb_traverse(ctx->tdb, idmap_tdb_dump_one_entry, data);
+ ctx->db->traverse_read(ctx->db, idmap_tdb_dump_one_entry, data);
if ( ! NT_STATUS_IS_OK(data->ret)) {
ret = data->ret;
{
NTSTATUS ret;
+ DEBUG(10, ("calling idmap_tdb_init\n"));
+
/* FIXME: bad hack to actually register also the alloc_tdb module without changining configure.in */
ret = idmap_alloc_tdb_init();
if (! NT_STATUS_IS_OK(ret)) {
git.samba.org / amitay / samba.git / blobdiff