s3:idmap_tdb2: talloc_zero (instead of talloc) the idmap_tdb2_context
[amitay/samba.git] / source3 / winbindd / idmap_tdb2.c
index 1db880a48b14bf0e104031f88c4a2130ef7ef3ef..9e29f5822d778b2249a3250823f38933a301811f 100644 (file)
 #undef DBGC_CLASS
 #define DBGC_CLASS DBGC_IDMAP
 
+struct idmap_tdb2_context {
+       const char *script; /* script to provide idmaps */
+};
+
 /* High water mark keys */
 #define HWM_GROUP  "GROUP HWM"
 #define HWM_USER   "USER HWM"
 
-static struct idmap_tdb2_state {
-       /* User and group id pool */
-       uid_t low_uid, high_uid;               /* Range of uids to allocate */
-       gid_t low_gid, high_gid;               /* Range of gids to allocate */
-       const char *idmap_script;
-} idmap_tdb2_state;
-
 
 static NTSTATUS idmap_tdb2_new_mapping(struct idmap_domain *dom,
                                       struct id_map *map);
@@ -54,49 +51,46 @@ static NTSTATUS idmap_tdb2_new_mapping(struct idmap_domain *dom,
 /* handle to the permanent tdb */
 static struct db_context *idmap_tdb2;
 
-static NTSTATUS idmap_tdb2_alloc_load(void);
 
-static NTSTATUS idmap_tdb2_load_ranges(void)
+/*
+ * check and initialize high/low water marks in the db
+ */
+static NTSTATUS idmap_tdb2_init_hwm(struct idmap_domain *dom)
 {
-       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;
-       }
+       uint32 low_id;
 
-       idmap_tdb2_state.low_uid = low_uid;
-       idmap_tdb2_state.high_uid = high_uid;
-       idmap_tdb2_state.low_gid = low_gid;
-       idmap_tdb2_state.high_gid = high_gid;
+       /* Create high water marks for group and user id */
 
-       if (idmap_tdb2_state.high_uid <= idmap_tdb2_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;
+       low_id = dbwrap_fetch_int32(idmap_tdb2, HWM_USER);
+       if ((low_id == -1) || (low_id < dom->low_id)) {
+               if (!NT_STATUS_IS_OK(dbwrap_trans_store_int32(
+                                            idmap_tdb2, HWM_USER,
+                                            dom->low_id))) {
+                       DEBUG(0, ("Unable to initialise user hwm in idmap "
+                                 "database\n"));
+                       return NT_STATUS_INTERNAL_DB_ERROR;
+               }
        }
 
-       if (idmap_tdb2_state.high_gid <= idmap_tdb2_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;
+       low_id = dbwrap_fetch_int32(idmap_tdb2, HWM_GROUP);
+       if ((low_id == -1) || (low_id < dom->low_id)) {
+               if (!NT_STATUS_IS_OK(dbwrap_trans_store_int32(
+                                            idmap_tdb2, HWM_GROUP,
+                                            dom->low_id))) {
+                       DEBUG(0, ("Unable to initialise group hwm in idmap "
+                                 "database\n"));
+                       return NT_STATUS_INTERNAL_DB_ERROR;
+               }
        }
 
        return NT_STATUS_OK;
 }
 
+
 /*
   open the permanent tdb
  */
-static NTSTATUS idmap_tdb2_open_db(void)
+static NTSTATUS idmap_tdb2_open_db(struct idmap_domain *dom)
 {
        char *db_path;
 
@@ -123,60 +117,7 @@ static NTSTATUS idmap_tdb2_open_db(void)
                return NT_STATUS_UNSUCCESSFUL;
        }
 
-       /* load the ranges and high/low water marks */
-       return idmap_tdb2_alloc_load();
-}
-
-
-/*
-  load the idmap allocation ranges and high/low water marks
-*/
-static NTSTATUS idmap_tdb2_alloc_load(void)
-{
-       NTSTATUS status;
-       uint32 low_id;
-
-       /* see if a idmap script is configured */
-       idmap_tdb2_state.idmap_script = lp_parm_const_string(-1, "idmap",
-                                                            "script", NULL);
-
-       if (idmap_tdb2_state.idmap_script) {
-               DEBUG(1, ("using idmap script '%s'\n",
-                         idmap_tdb2_state.idmap_script));
-       }
-
-       /* load ranges */
-
-       status = idmap_tdb2_load_ranges();
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
-       }
-
-       /* Create high water marks for group and user id */
-
-       low_id = dbwrap_fetch_int32(idmap_tdb2, HWM_USER);
-       if ((low_id == -1) || (low_id < idmap_tdb2_state.low_uid)) {
-               if (!NT_STATUS_IS_OK(dbwrap_trans_store_int32(
-                                            idmap_tdb2, HWM_USER,
-                                            idmap_tdb2_state.low_uid))) {
-                       DEBUG(0, ("Unable to initialise user hwm in idmap "
-                                 "database\n"));
-                       return NT_STATUS_INTERNAL_DB_ERROR;
-               }
-       }
-
-       low_id = dbwrap_fetch_int32(idmap_tdb2, HWM_GROUP);
-       if ((low_id == -1) || (low_id < idmap_tdb2_state.low_gid)) {
-               if (!NT_STATUS_IS_OK(dbwrap_trans_store_int32(
-                                            idmap_tdb2, HWM_GROUP,
-                                            idmap_tdb2_state.low_gid))) {
-                       DEBUG(0, ("Unable to initialise group hwm in idmap "
-                                 "database\n"));
-                       return NT_STATUS_INTERNAL_DB_ERROR;
-               }
-       }
-
-       return NT_STATUS_OK;
+       return idmap_tdb2_init_hwm(dom);
 }
 
 
@@ -237,7 +178,8 @@ done:
        return ret;
 }
 
-static NTSTATUS idmap_tdb2_allocate_id(struct unixid *xid)
+static NTSTATUS idmap_tdb2_allocate_id(struct idmap_domain *dom,
+                                      struct unixid *xid)
 {
        const char *hwmkey;
        const char *hwmtype;
@@ -246,7 +188,7 @@ static NTSTATUS idmap_tdb2_allocate_id(struct unixid *xid)
        NTSTATUS status;
        struct idmap_tdb2_allocate_id_context state;
 
-       status = idmap_tdb2_open_db();
+       status = idmap_tdb2_open_db(dom);
        NT_STATUS_NOT_OK_RETURN(status);
 
        /* Get current high water mark */
@@ -255,13 +197,11 @@ static NTSTATUS idmap_tdb2_allocate_id(struct unixid *xid)
        case ID_TYPE_UID:
                hwmkey = HWM_USER;
                hwmtype = "UID";
-               high_hwm = idmap_tdb2_state.high_uid;
                break;
 
        case ID_TYPE_GID:
                hwmkey = HWM_GROUP;
                hwmtype = "GID";
-               high_hwm = idmap_tdb2_state.high_gid;
                break;
 
        default:
@@ -269,6 +209,8 @@ static NTSTATUS idmap_tdb2_allocate_id(struct unixid *xid)
                return NT_STATUS_INVALID_PARAMETER;
        }
 
+       high_hwm = dom->high_id;
+
        state.hwm = hwm;
        state.high_hwm = high_hwm;
        state.hwmtype = hwmtype;
@@ -306,7 +248,7 @@ static NTSTATUS idmap_tdb2_get_new_id(struct idmap_domain *dom,
                return NT_STATUS_NOT_IMPLEMENTED;
        }
 
-       ret = idmap_tdb2_allocate_id(id);
+       ret = idmap_tdb2_allocate_id(dom, id);
 
        return ret;
 }
@@ -314,10 +256,6 @@ static NTSTATUS idmap_tdb2_get_new_id(struct idmap_domain *dom,
 /*
   IDMAP MAPPING TDB BACKEND
 */
-struct idmap_tdb2_context {
-       uint32_t filter_low_id;
-       uint32_t filter_high_id;
-};
 
 /*
   Initialise idmap database. 
@@ -329,47 +267,20 @@ static NTSTATUS idmap_tdb2_db_init(struct idmap_domain *dom,
        struct idmap_tdb2_context *ctx;
        NTSTATUS status;
 
-       status = idmap_tdb2_open_db();
-       NT_STATUS_NOT_OK_RETURN(status);
-
-       ctx = talloc(dom, struct idmap_tdb2_context);
+       ctx = talloc_zero(dom, struct idmap_tdb2_context);
        if ( ! ctx) {
                DEBUG(0, ("Out of memory!\n"));
                return NT_STATUS_NO_MEMORY;
        }
 
        if (strequal(dom->name, "*")) {
-               uid_t low_uid = 0;
-               uid_t high_uid = 0;
-               gid_t low_gid = 0;
-               gid_t high_gid = 0;
-
-               ctx->filter_low_id = 0;
-               ctx->filter_high_id = 0;
-
-               if (lp_idmap_uid(&low_uid, &high_uid)) {
-                       ctx->filter_low_id = low_uid;
-                       ctx->filter_high_id = high_uid;
-               } else {
-                       DEBUG(3, ("Warning: 'idmap uid' not set!\n"));
-               }
-
-               if (lp_idmap_gid(&low_gid, &high_gid)) {
-                       if ((low_gid != low_uid) || (high_gid != high_uid)) {
-                               DEBUG(1, ("Warning: 'idmap uid' and 'idmap gid'"
-                                     " ranges do not agree -- building "
-                                     "intersection\n"));
-                               ctx->filter_low_id = MAX(ctx->filter_low_id,
-                                                        low_gid);
-                               ctx->filter_high_id = MIN(ctx->filter_high_id,
-                                                         high_gid);
-                       }
-               } else {
-                       DEBUG(3, ("Warning: 'idmap gid' not set!\n"));
+               ctx->script = lp_parm_const_string(-1, "idmap", "script", NULL);
+               if (ctx->script) {
+                       DEBUG(1, ("using idmap script '%s'\n", ctx->script));
                }
        } else {
                char *config_option = NULL;
-               const char *range;
+
                config_option = talloc_asprintf(ctx, "idmap config %s", dom->name);
                if ( ! config_option) {
                        DEBUG(0, ("Out of memory!\n"));
@@ -377,24 +288,21 @@ static NTSTATUS idmap_tdb2_db_init(struct idmap_domain *dom,
                        goto failed;
                }
 
-               range = lp_parm_const_string(-1, config_option, "range", NULL);
-               if (( ! range) ||
-                   (sscanf(range, "%u - %u", &ctx->filter_low_id, &ctx->filter_high_id) != 2))
-               {
-                       ctx->filter_low_id = 0;
-                       ctx->filter_high_id = 0;
+               ctx->script = lp_parm_const_string(-1, config_option, "script", NULL);
+               if (ctx->script) {
+                       DEBUG(1, ("using idmap script '%s'\n", ctx->script));
                }
 
                talloc_free(config_option);
        }
 
-       if (ctx->filter_low_id > ctx->filter_high_id) {
-               ctx->filter_low_id = 0;
-               ctx->filter_high_id = 0;
-       }
-
        dom->private_data = ctx;
 
+       ret = idmap_tdb2_open_db(dom);
+       if (!NT_STATUS_IS_OK(ret)) {
+               goto failed;
+       }
+
        return NT_STATUS_OK;
 
 failed:
@@ -476,7 +384,7 @@ static NTSTATUS idmap_tdb2_script(struct idmap_tdb2_context *ctx, struct id_map
        char line[64];
        unsigned long v;
 
-       cmd = talloc_asprintf(ctx, "%s ", idmap_tdb2_state.idmap_script);
+       cmd = talloc_asprintf(ctx, "%s ", ctx->script);
        NT_STATUS_HAVE_NO_MEMORY(cmd);  
 
        va_start(ap, fmt);
@@ -507,12 +415,12 @@ static NTSTATUS idmap_tdb2_script(struct idmap_tdb2_context *ctx, struct id_map
        } else if (strncmp(line, "SID:S-", 6) == 0) {
                if (!string_to_sid(map->sid, &line[4])) {
                        DEBUG(0,("Bad SID in '%s' from idmap script %s\n",
-                                line, idmap_tdb2_state.idmap_script));
+                                line, ctx->script));
                        return NT_STATUS_NONE_MAPPED;                   
                }
        } else {
                DEBUG(0,("Bad reply '%s' from idmap script %s\n",
-                        line, idmap_tdb2_state.idmap_script));
+                        line, ctx->script));
                return NT_STATUS_NONE_MAPPED;
        }
 
@@ -524,25 +432,28 @@ static NTSTATUS idmap_tdb2_script(struct idmap_tdb2_context *ctx, struct id_map
 /*
   Single id to sid lookup function. 
 */
-static NTSTATUS idmap_tdb2_id_to_sid(struct idmap_tdb2_context *ctx, struct id_map *map)
+static NTSTATUS idmap_tdb2_id_to_sid(struct idmap_domain *dom, struct id_map *map)
 {
        NTSTATUS ret;
        TDB_DATA data;
        char *keystr;
        NTSTATUS status;
+       struct idmap_tdb2_context *ctx;
 
-       status = idmap_tdb2_open_db();
-       NT_STATUS_NOT_OK_RETURN(status);
 
-       if (!ctx || !map) {
+       if (!dom || !map) {
                return NT_STATUS_INVALID_PARAMETER;
        }
 
+       status = idmap_tdb2_open_db(dom);
+       NT_STATUS_NOT_OK_RETURN(status);
+
+       ctx = talloc_get_type(dom->private_data, struct idmap_tdb2_context);
+
        /* apply filters before checking */
-       if ((ctx->filter_low_id && (map->xid.id < ctx->filter_low_id)) ||
-           (ctx->filter_high_id && (map->xid.id > ctx->filter_high_id))) {
+       if (!idmap_unix_id_is_in_range(map->xid.id, dom)) {
                DEBUG(5, ("Requested id (%u) out of range (%u - %u). Filtered!\n",
-                               map->xid.id, ctx->filter_low_id, ctx->filter_high_id));
+                               map->xid.id, dom->low_id, dom->high_id));
                return NT_STATUS_NONE_MAPPED;
        }
 
@@ -580,7 +491,7 @@ static NTSTATUS idmap_tdb2_id_to_sid(struct idmap_tdb2_context *ctx, struct id_m
                struct idmap_tdb2_set_mapping_context store_state;
 
                DEBUG(10,("Record %s not found\n", keystr));
-               if (idmap_tdb2_state.idmap_script == NULL) {
+               if (ctx->script == NULL) {
                        ret = NT_STATUS_NONE_MAPPED;
                        goto done;
                }
@@ -625,17 +536,20 @@ done:
 /*
  Single sid to id lookup function. 
 */
-static NTSTATUS idmap_tdb2_sid_to_id(struct idmap_tdb2_context *ctx, struct id_map *map)
+static NTSTATUS idmap_tdb2_sid_to_id(struct idmap_domain *dom, struct id_map *map)
 {
        NTSTATUS ret;
        TDB_DATA data;
        char *keystr;
        unsigned long rec_id = 0;
+       struct idmap_tdb2_context *ctx;
        TALLOC_CTX *tmp_ctx = talloc_stackframe();
 
-       ret = idmap_tdb2_open_db();
+       ret = idmap_tdb2_open_db(dom);
        NT_STATUS_NOT_OK_RETURN(ret);
 
+       ctx = talloc_get_type(dom->private_data, struct idmap_tdb2_context);
+
        keystr = sid_string_talloc(tmp_ctx, map->sid);
        if (keystr == NULL) {
                DEBUG(0, ("Out of memory!\n"));
@@ -653,7 +567,7 @@ static NTSTATUS idmap_tdb2_sid_to_id(struct idmap_tdb2_context *ctx, struct id_m
 
                DEBUG(10,(__location__ " Record %s not found\n", keystr));
 
-               if (idmap_tdb2_state.idmap_script == NULL) {
+               if (ctx->script == NULL) {
                        ret = NT_STATUS_NONE_MAPPED;
                        goto done;
                }
@@ -665,14 +579,10 @@ static NTSTATUS idmap_tdb2_sid_to_id(struct idmap_tdb2_context *ctx, struct id_m
                }
 
                /* apply filters before returning result */
-               if ((ctx->filter_low_id
-                    && (map->xid.id < ctx->filter_low_id)) ||
-                   (ctx->filter_high_id
-                    && (map->xid.id > ctx->filter_high_id))) {
+               if (!idmap_unix_id_is_in_range(map->xid.id, dom)) {
                        DEBUG(5, ("Script returned id (%u) out of range "
                                  "(%u - %u). Filtered!\n",
-                                 map->xid.id,
-                                 ctx->filter_low_id, ctx->filter_high_id));
+                                 map->xid.id, dom->low_id, dom->high_id));
                        ret = NT_STATUS_NONE_MAPPED;
                        goto done;
                }
@@ -713,10 +623,9 @@ static NTSTATUS idmap_tdb2_sid_to_id(struct idmap_tdb2_context *ctx, struct id_m
        }
 
        /* apply filters before returning result */
-       if ((ctx->filter_low_id && (map->xid.id < ctx->filter_low_id)) ||
-           (ctx->filter_high_id && (map->xid.id > ctx->filter_high_id))) {
+       if (!idmap_unix_id_is_in_range(map->xid.id, dom)) {
                DEBUG(5, ("Requested id (%u) out of range (%u - %u). Filtered!\n",
-                               map->xid.id, ctx->filter_low_id, ctx->filter_high_id));
+                               map->xid.id, dom->low_id, dom->high_id));
                ret = NT_STATUS_NONE_MAPPED;
        }
 
@@ -730,7 +639,6 @@ done:
 */
 static NTSTATUS idmap_tdb2_unixids_to_sids(struct idmap_domain *dom, struct id_map **ids)
 {
-       struct idmap_tdb2_context *ctx;
        NTSTATUS ret;
        int i;
 
@@ -739,10 +647,8 @@ static NTSTATUS idmap_tdb2_unixids_to_sids(struct idmap_domain *dom, struct id_m
                ids[i]->status = ID_UNKNOWN;
        }
 
-       ctx = talloc_get_type(dom->private_data, struct idmap_tdb2_context);
-
        for (i = 0; ids[i]; i++) {
-               ret = idmap_tdb2_id_to_sid(ctx, ids[i]);
+               ret = idmap_tdb2_id_to_sid(dom, ids[i]);
                if ( ! NT_STATUS_IS_OK(ret)) {
 
                        /* if it is just a failed mapping continue */
@@ -782,7 +688,6 @@ static NTSTATUS idmap_tdb2_sids_to_unixids_action(struct db_context *db,
 {
        struct idmap_tdb2_sids_to_unixids_context *state;
        int i;
-       struct idmap_tdb2_context *ctx;
        NTSTATUS ret = NT_STATUS_OK;
 
        state = (struct idmap_tdb2_sids_to_unixids_context *)private_data;
@@ -792,10 +697,6 @@ static NTSTATUS idmap_tdb2_sids_to_unixids_action(struct db_context *db,
                   state->dom->name,
                   state->allocate_unmapped ? "yes" : "no"));
 
-       ctx = talloc_get_type(state->dom->private_data,
-                             struct idmap_tdb2_context);
-
-
        for (i = 0; state->ids[i]; i++) {
                if ((state->ids[i]->status == ID_UNKNOWN) ||
                    /* retry if we could not map in previous run: */
@@ -803,7 +704,7 @@ static NTSTATUS idmap_tdb2_sids_to_unixids_action(struct db_context *db,
                {
                        NTSTATUS ret2;
 
-                       ret2 = idmap_tdb2_sid_to_id(ctx, state->ids[i]);
+                       ret2 = idmap_tdb2_sid_to_id(state->dom, state->ids[i]);
                        if (!NT_STATUS_IS_OK(ret2)) {
 
                                /* if it is just a failed mapping, continue */
@@ -854,7 +755,7 @@ static NTSTATUS idmap_tdb2_sids_to_unixids(struct idmap_domain *dom, struct id_m
 
        ret = idmap_tdb2_sids_to_unixids_action(idmap_tdb2, &state);
 
-       if (NT_STATUS_EQUAL(ret, STATUS_SOME_UNMAPPED)) {
+       if (NT_STATUS_EQUAL(ret, STATUS_SOME_UNMAPPED) && !dom->read_only) {
                state.allocate_unmapped = true;
                ret = dbwrap_trans_do(idmap_tdb2,
                                      idmap_tdb2_sids_to_unixids_action,