diff options
author | Volker Lendecke <vl@samba.org> | 2014-02-12 14:47:26 +0000 |
---|---|---|
committer | Michael Adam <obnox@samba.org> | 2014-02-12 21:47:51 +0100 |
commit | 69594dfb8cfa35612ff9ce96f9669d7ed94ad578 (patch) | |
tree | 473fe577289fe7cef1a0360c167f2d8e07feec00 /source3/auth/auth_samba4.c | |
parent | f0a385f35f9528b8f1f0c829065562b2d90b9850 (diff) | |
download | samba-69594dfb8cfa35612ff9ce96f9669d7ed94ad578.tar.gz |
auth: Make new_server_id_task() static to auth_samba4
This is not used in other parts of source3, so this patch improves
modularity and isolation of features.
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'source3/auth/auth_samba4.c')
-rw-r--r-- | source3/auth/auth_samba4.c | 42 |
1 files changed, 42 insertions, 0 deletions
diff --git a/source3/auth/auth_samba4.c b/source3/auth/auth_samba4.c index 0c2beace804..901acf9f8ac 100644 --- a/source3/auth/auth_samba4.c +++ b/source3/auth/auth_samba4.c @@ -31,6 +31,48 @@ #undef DBGC_CLASS #define DBGC_CLASS DBGC_AUTH +static struct idr_context *task_id_tree; + +static int free_task_id(struct server_id *server_id) +{ + idr_remove(task_id_tree, server_id->task_id); + return 0; +} + +/* Return a server_id with a unique task_id element. Free the + * returned pointer to de-allocate the task_id via a talloc destructor + * (ie, use talloc_free()) */ +static struct server_id *new_server_id_task(TALLOC_CTX *mem_ctx) +{ + struct server_id *server_id; + int task_id; + if (!task_id_tree) { + task_id_tree = idr_init(NULL); + if (!task_id_tree) { + return NULL; + } + } + + server_id = talloc(mem_ctx, struct server_id); + + if (!server_id) { + return NULL; + } + *server_id = procid_self(); + + /* 0 is the default server_id, so we need to start with 1 */ + task_id = idr_get_new_above(task_id_tree, server_id, 1, INT32_MAX); + + if (task_id == -1) { + talloc_free(server_id); + return NULL; + } + + talloc_set_destructor(server_id, free_task_id); + server_id->task_id = task_id; + return server_id; +} + /* * This module is not an ordinary authentication module. It is really * a way to redirect the whole authentication and authorization stack |