From 145fb99b6e0351d92ffd2f85be77b426c89df59b Mon Sep 17 00:00:00 2001 From: Tim Dijkstra Date: Tue, 12 Dec 2006 21:24:07 +0000 Subject: commit.msg --- bus/bus.c | 22 ++-------------------- bus/bus.h | 3 --- bus/connection.c | 9 ++------- bus/policy.c | 5 ++--- bus/policy.h | 1 - 5 files changed, 6 insertions(+), 34 deletions(-) (limited to 'bus') diff --git a/bus/bus.c b/bus/bus.c index efbd0232..562eb983 100644 --- a/bus/bus.c +++ b/bus/bus.c @@ -34,6 +34,7 @@ #include #include #include +#include struct BusContext { @@ -50,7 +51,6 @@ struct BusContext BusRegistry *registry; BusPolicy *policy; BusMatchmaker *matchmaker; - DBusUserDatabase *user_database; BusLimits limits; unsigned int fork : 1; }; @@ -596,13 +596,6 @@ bus_context_new (const DBusString *config_file, if (!dbus_server_allocate_data_slot (&server_data_slot)) _dbus_assert_not_reached ("second ref of server data slot failed"); - context->user_database = _dbus_user_database_new (); - if (context->user_database == NULL) - { - BUS_SET_OOM (error); - goto failed; - } - /* Note that we don't know whether the print_addr_fd is * one of the sockets we're using to listen on, or some * other random thing. But I think the answer is "don't do @@ -800,7 +793,7 @@ bus_context_reload_config (BusContext *context, dbus_bool_t ret; /* Flush the user database cache */ - _dbus_user_database_flush(context->user_database); + _dbus_user_database_flush_system (); ret = FALSE; _dbus_string_init_const (&config_file, context->config_file); @@ -951,10 +944,6 @@ bus_context_unref (BusContext *context) dbus_free (context->pidfile); } - - if (context->user_database != NULL) - _dbus_user_database_unref (context->user_database); - dbus_free (context); dbus_server_free_data_slot (&server_data_slot); @@ -1004,18 +993,11 @@ bus_context_get_loop (BusContext *context) return context->loop; } -DBusUserDatabase* -bus_context_get_user_database (BusContext *context) -{ - return context->user_database; -} - dbus_bool_t bus_context_allow_user (BusContext *context, unsigned long uid) { return bus_policy_allow_user (context->policy, - context->user_database, uid); } diff --git a/bus/bus.h b/bus/bus.h index 527bfddc..77d764db 100644 --- a/bus/bus.h +++ b/bus/bus.h @@ -29,7 +29,6 @@ #include #include #include -#include typedef struct BusActivation BusActivation; typedef struct BusConnections BusConnections; @@ -86,8 +85,6 @@ BusConnections* bus_context_get_connections (BusContext BusActivation* bus_context_get_activation (BusContext *context); BusMatchmaker* bus_context_get_matchmaker (BusContext *context); DBusLoop* bus_context_get_loop (BusContext *context); -DBusUserDatabase* bus_context_get_user_database (BusContext *context); - dbus_bool_t bus_context_allow_user (BusContext *context, unsigned long uid); BusPolicy* bus_context_get_policy (BusContext *context); diff --git a/bus/connection.c b/bus/connection.c index df888f47..d063afca 100644 --- a/bus/connection.c +++ b/bus/connection.c @@ -31,6 +31,7 @@ #include #include #include +#include static void bus_connection_remove_transactions (DBusConnection *connection); @@ -778,24 +779,18 @@ bus_connection_get_groups (DBusConnection *connection, { BusConnectionData *d; unsigned long uid; - DBusUserDatabase *user_database; d = BUS_CONNECTION_DATA (connection); _dbus_assert (d != NULL); - user_database = bus_context_get_user_database (d->connections->context); - *groups = NULL; *n_groups = 0; if (dbus_connection_get_unix_user (connection, &uid)) { - if (!_dbus_user_database_get_groups (user_database, - uid, groups, n_groups, - error)) + if (!_dbus_groups_from_uid (uid, groups, n_groups)) { - _DBUS_ASSERT_ERROR_IS_SET (error); _dbus_verbose ("Did not get any groups for UID %lu\n", uid); return FALSE; diff --git a/bus/policy.c b/bus/policy.c index c0244bdc..7b4c5f26 100644 --- a/bus/policy.c +++ b/bus/policy.c @@ -28,6 +28,7 @@ #include #include #include +#include BusPolicyRule* bus_policy_rule_new (BusPolicyRuleType type, @@ -438,7 +439,6 @@ list_allows_user (dbus_bool_t def, dbus_bool_t bus_policy_allow_user (BusPolicy *policy, - DBusUserDatabase *user_database, unsigned long uid) { dbus_bool_t allowed; @@ -446,8 +446,7 @@ bus_policy_allow_user (BusPolicy *policy, int n_group_ids; /* On OOM or error we always reject the user */ - if (!_dbus_user_database_get_groups (user_database, - uid, &group_ids, &n_group_ids, NULL)) + if (!_dbus_groups_from_uid (uid, &group_ids, &n_group_ids)) { _dbus_verbose ("Did not get any groups for UID %lu\n", uid); diff --git a/bus/policy.h b/bus/policy.h index 94b48e6f..0a944ff1 100644 --- a/bus/policy.h +++ b/bus/policy.h @@ -113,7 +113,6 @@ BusClientPolicy* bus_policy_create_client_policy (BusPolicy *policy, DBusConnection *connection, DBusError *error); dbus_bool_t bus_policy_allow_user (BusPolicy *policy, - DBusUserDatabase *user_database, unsigned long uid); dbus_bool_t bus_policy_append_default_rule (BusPolicy *policy, BusPolicyRule *rule); -- cgit