X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=blobdiff_plain;f=src%2Fhostname%2Fhostnamectl.c;h=8d8dc3d7f357e73a24def4fe363f08f28c45d168;hp=917fac2466c940461f42ca6a3719b14a46838999;hb=fc7689bc9ccf9b94f9a9f666f84c79b8f5b84791;hpb=fda2c5d28bdf5c6ac1ff1ba504fa884c48ed1439 diff --git a/src/hostname/hostnamectl.c b/src/hostname/hostnamectl.c index 917fac246..8d8dc3d7f 100644 --- a/src/hostname/hostnamectl.c +++ b/src/hostname/hostnamectl.c @@ -28,7 +28,10 @@ #include #include -#include "dbus-common.h" +#include "sd-bus.h" + +#include "bus-util.h" +#include "bus-error.h" #include "util.h" #include "spawn-polkit-agent.h" #include "build.h" @@ -38,33 +41,19 @@ #include "virt.h" #include "fileio.h" -static enum transport { - TRANSPORT_NORMAL, - TRANSPORT_SSH, - TRANSPORT_POLKIT -} arg_transport = TRANSPORT_NORMAL; static bool arg_ask_password = true; -static const char *arg_host = NULL; -static bool arg_set_transient = false; -static bool arg_set_pretty = false; -static bool arg_set_static = false; - -static void polkit_agent_open_if_enabled(void) { - - /* Open the polkit agent as a child process if necessary */ - - if (!arg_ask_password) - return; - - polkit_agent_open(); -} +static BusTransport arg_transport = BUS_TRANSPORT_LOCAL; +static char *arg_host = NULL; +static bool arg_transient = false; +static bool arg_pretty = false; +static bool arg_static = false; typedef struct StatusInfo { - const char *hostname; - const char *static_hostname; - const char *pretty_hostname; - const char *icon_name; - const char *chassis; + char *hostname; + char *static_hostname; + char *pretty_hostname; + char *icon_name; + char *chassis; } StatusInfo; static void print_status_info(StatusInfo *i) { @@ -79,14 +68,18 @@ static void print_status_info(StatusInfo *i) { printf(" Static hostname: %s\n", strna(i->static_hostname)); - if (!streq_ptr(i->hostname, i->static_hostname)) + if (!isempty(i->pretty_hostname) && + !streq_ptr(i->pretty_hostname, i->static_hostname)) + printf(" Pretty hostname: %s\n", + strna(i->pretty_hostname)); + + if (!isempty(i->hostname) && + !streq_ptr(i->hostname, i->static_hostname)) printf("Transient hostname: %s\n", strna(i->hostname)); - printf(" Pretty hostname: %s\n" - " Icon name: %s\n" + printf(" Icon name: %s\n" " Chassis: %s\n", - strna(i->pretty_hostname), strna(i->icon_name), strna(i->chassis)); @@ -105,6 +98,8 @@ static void print_status_info(StatusInfo *i) { "PRETTY_NAME", &pretty_name, "CPE_NAME", &cpe_name, NULL); + if (r < 0) + log_warning("Failed to read /etc/os-release: %s", strerror(-r)); if (!isempty(pretty_name)) printf(" Operating System: %s\n", pretty_name); @@ -118,125 +113,105 @@ static void print_status_info(StatusInfo *i) { } -static int status_property(const char *name, DBusMessageIter *iter, StatusInfo *i) { - assert(name); - assert(iter); - - switch (dbus_message_iter_get_arg_type(iter)) { - - case DBUS_TYPE_STRING: { - const char *s; - - dbus_message_iter_get_basic(iter, &s); - if (!isempty(s)) { - if (streq(name, "Hostname")) - i->hostname = s; - if (streq(name, "StaticHostname")) - i->static_hostname = s; - if (streq(name, "PrettyHostname")) - i->pretty_hostname = s; - if (streq(name, "IconName")) - i->icon_name = s; - if (streq(name, "Chassis")) - i->chassis = s; - } - break; - } - } - - return 0; -} - -static int show_status(DBusConnection *bus, char **args, unsigned n) { - _cleanup_dbus_message_unref_ DBusMessage *reply = NULL; - const char *interface = ""; +static int show_one_name(sd_bus *bus, const char* attr) { + _cleanup_bus_message_unref_ sd_bus_message *reply = NULL; + _cleanup_bus_error_free_ sd_bus_error error = SD_BUS_ERROR_NULL; + const char *s; int r; - DBusMessageIter iter, sub, sub2, sub3; - StatusInfo info; - - assert(args); - r = bus_method_call_with_reply( + r = sd_bus_get_property( bus, "org.freedesktop.hostname1", "/org/freedesktop/hostname1", - "org.freedesktop.DBus.Properties", - "GetAll", - &reply, - NULL, - DBUS_TYPE_STRING, &interface, - DBUS_TYPE_INVALID); + "org.freedesktop.hostname1", + attr, + &error, &reply, "s"); + if (r < 0) { + log_error("Could not get property: %s", bus_error_message(&error, -r)); + return r; + } + + r = sd_bus_message_read(reply, "s", &s); if (r < 0) return r; - if (!dbus_message_iter_init(reply, &iter) || - dbus_message_iter_get_arg_type(&iter) != DBUS_TYPE_ARRAY || - dbus_message_iter_get_element_type(&iter) != DBUS_TYPE_DICT_ENTRY) { - log_error("Failed to parse reply."); - return -EIO; - } + printf("%s\n", s); - zero(info); - dbus_message_iter_recurse(&iter, &sub); + return 0; +} - while (dbus_message_iter_get_arg_type(&sub) != DBUS_TYPE_INVALID) { - const char *name; +static int show_all_names(sd_bus *bus) { + StatusInfo info = {}; + const struct bus_properties_map map[] = { + { "s", "Hostname", &info.hostname }, + { "s", "StaticHostname", &info.static_hostname }, + { "s", "PrettyHostname", &info.pretty_hostname }, + { "s", "IconName", &info.icon_name }, + { "s", "Chassis", &info.chassis }, + {} + }; + int r; - if (dbus_message_iter_get_arg_type(&sub) != DBUS_TYPE_DICT_ENTRY) { - log_error("Failed to parse reply."); - return -EIO; - } + r = bus_map_all_properties(bus, + "org.freedesktop.hostname1", + "/org/freedesktop/hostname1", + map); + if (r < 0) + goto fail; - dbus_message_iter_recurse(&sub, &sub2); + print_status_info(&info); - if (bus_iter_get_basic_and_next(&sub2, DBUS_TYPE_STRING, &name, true) < 0) { - log_error("Failed to parse reply."); - return -EIO; - } +fail: + free(info.hostname); + free(info.static_hostname); + free(info.pretty_hostname); + free(info.icon_name); + free(info.chassis); + return 0; +} - if (dbus_message_iter_get_arg_type(&sub2) != DBUS_TYPE_VARIANT) { - log_error("Failed to parse reply."); - return -EIO; - } +static int show_status(sd_bus *bus, char **args, unsigned n) { + assert(args); - dbus_message_iter_recurse(&sub2, &sub3); + if (arg_pretty || arg_static || arg_transient) { + const char *attr; - r = status_property(name, &sub3, &info); - if (r < 0) { - log_error("Failed to parse reply."); - return r; + if (!!arg_static + !!arg_pretty + !!arg_transient > 1) { + log_error("Cannot query more than one name type at a time"); + return -EINVAL; } - dbus_message_iter_next(&sub); - } + attr = arg_pretty ? "PrettyHostname" : + arg_static ? "StaticHostname" : "Hostname"; - print_status_info(&info); - return 0; + return show_one_name(bus, attr); + } else + return show_all_names(bus); } -static char* hostname_simplify(char *s) { - char *p, *d; - - for (p = s, d = s; *p; p++) { - if ((*p >= 'a' && *p <= 'z') || - (*p >= '0' && *p <= '9') || - *p == '-' || *p == '_') - *(d++) = *p; - else if (*p >= 'A' && *p <= 'Z') - *(d++) = *p - 'A' + 'a'; - else if (*p == ' ') - *(d++) = '-'; - } +static int set_simple_string(sd_bus *bus, const char *method, const char *value) { + _cleanup_bus_error_free_ sd_bus_error error = SD_BUS_ERROR_NULL; + static bool first=true; + int r = 0; - *d = 0; + if (first && arg_ask_password) + polkit_agent_open(); + first = false; - strshorten(s, HOST_NAME_MAX); - return s; + r = sd_bus_call_method( + bus, + "org.freedesktop.hostname1", + "/org/freedesktop/hostname1", + "org.freedesktop.hostname1", + method, + &error, NULL, + "sb", value, arg_ask_password); + if (r < 0) + log_error("Could not set property: %s", bus_error_message(&error, -r)); + return r; } -static int set_hostname(DBusConnection *bus, char **args, unsigned n) { - _cleanup_dbus_message_unref_ DBusMessage *reply = NULL; - dbus_bool_t interactive = true; +static int set_hostname(sd_bus *bus, char **args, unsigned n) { _cleanup_free_ char *h = NULL; const char *hostname = args[1]; int r; @@ -244,9 +219,10 @@ static int set_hostname(DBusConnection *bus, char **args, unsigned n) { assert(args); assert(n == 2); - polkit_agent_open_if_enabled(); + if (!arg_pretty && !arg_static && !arg_transient) + arg_pretty = arg_static = arg_transient = true; - if (arg_set_pretty) { + if (arg_pretty) { const char *p; /* If the passed hostname is already valid, then @@ -255,69 +231,32 @@ static int set_hostname(DBusConnection *bus, char **args, unsigned n) { * just set the passed hostname as static/dynamic * hostname. */ - if (hostname_is_valid(hostname)) + h = strdup(hostname); + if (!h) + return log_oom(); + + hostname_cleanup(h, true); + + if (arg_static && streq(h, hostname)) p = ""; else { p = hostname; - - h = strdup(hostname); - if (!h) - return log_oom(); - - hostname = hostname_simplify(h); + hostname = h; } - r = bus_method_call_with_reply( - bus, - "org.freedesktop.hostname1", - "/org/freedesktop/hostname1", - "org.freedesktop.hostname1", - "SetPrettyHostname", - &reply, - NULL, - DBUS_TYPE_STRING, &p, - DBUS_TYPE_BOOLEAN, &interactive, - DBUS_TYPE_INVALID); + r = set_simple_string(bus, "SetPrettyHostname", p); if (r < 0) return r; - - dbus_message_unref(reply); - reply = NULL; } - if (arg_set_static) { - r = bus_method_call_with_reply( - bus, - "org.freedesktop.hostname1", - "/org/freedesktop/hostname1", - "org.freedesktop.hostname1", - "SetStaticHostname", - &reply, - NULL, - DBUS_TYPE_STRING, &hostname, - DBUS_TYPE_BOOLEAN, &interactive, - DBUS_TYPE_INVALID); - + if (arg_static) { + r = set_simple_string(bus, "SetStaticHostname", hostname); if (r < 0) return r; - - dbus_message_unref(reply); - reply = NULL; } - if (arg_set_transient) { - r = bus_method_call_with_reply( - bus, - "org.freedesktop.hostname1", - "/org/freedesktop/hostname1", - "org.freedesktop.hostname1", - "SetHostname", - &reply, - NULL, - DBUS_TYPE_STRING, &hostname, - DBUS_TYPE_BOOLEAN, &interactive, - DBUS_TYPE_INVALID); - + if (arg_transient) { + r = set_simple_string(bus, "SetHostname", hostname); if (r < 0) return r; } @@ -325,48 +264,18 @@ static int set_hostname(DBusConnection *bus, char **args, unsigned n) { return 0; } -static int set_icon_name(DBusConnection *bus, char **args, unsigned n) { - _cleanup_dbus_message_unref_ DBusMessage *reply = NULL; - dbus_bool_t interactive = true; - +static int set_icon_name(sd_bus *bus, char **args, unsigned n) { assert(args); assert(n == 2); - polkit_agent_open_if_enabled(); - - return bus_method_call_with_reply( - bus, - "org.freedesktop.hostname1", - "/org/freedesktop/hostname1", - "org.freedesktop.hostname1", - "SetIconName", - &reply, - NULL, - DBUS_TYPE_STRING, &args[1], - DBUS_TYPE_BOOLEAN, &interactive, - DBUS_TYPE_INVALID); + return set_simple_string(bus, "SetIconName", args[1]); } -static int set_chassis(DBusConnection *bus, char **args, unsigned n) { - _cleanup_dbus_message_unref_ DBusMessage *reply = NULL; - dbus_bool_t interactive = true; - +static int set_chassis(sd_bus *bus, char **args, unsigned n) { assert(args); assert(n == 2); - polkit_agent_open_if_enabled(); - - return bus_method_call_with_reply( - bus, - "org.freedesktop.hostname1", - "/org/freedesktop/hostname1", - "org.freedesktop.hostname1", - "SetChassis", - &reply, - NULL, - DBUS_TYPE_STRING, &args[1], - DBUS_TYPE_BOOLEAN, &interactive, - DBUS_TYPE_INVALID); + return set_simple_string(bus, "SetChasis", args[1]); } static int help(void) { @@ -379,7 +288,8 @@ static int help(void) { " --static Only set static hostname\n" " --pretty Only set pretty hostname\n" " --no-ask-password Do not prompt for password\n" - " -H --host=[USER@]HOST Operate on remote host\n\n" + " -H --host=[USER@]HOST Operate on remote host\n" + " -M --machine=CONTAINER Operate on local container\n\n" "Commands:\n" " status Show current hostname settings\n" " set-hostname NAME Set system hostname\n" @@ -395,19 +305,19 @@ static int parse_argv(int argc, char *argv[]) { enum { ARG_VERSION = 0x100, ARG_NO_ASK_PASSWORD, - ARG_SET_TRANSIENT, - ARG_SET_STATIC, - ARG_SET_PRETTY + ARG_TRANSIENT, + ARG_STATIC, + ARG_PRETTY }; static const struct option options[] = { { "help", no_argument, NULL, 'h' }, { "version", no_argument, NULL, ARG_VERSION }, - { "transient", no_argument, NULL, ARG_SET_TRANSIENT }, - { "static", no_argument, NULL, ARG_SET_STATIC }, - { "pretty", no_argument, NULL, ARG_SET_PRETTY }, + { "transient", no_argument, NULL, ARG_TRANSIENT }, + { "static", no_argument, NULL, ARG_STATIC }, + { "pretty", no_argument, NULL, ARG_PRETTY }, { "host", required_argument, NULL, 'H' }, - { "privileged", no_argument, NULL, 'P' }, + { "machine", required_argument, NULL, 'M' }, { "no-ask-password", no_argument, NULL, ARG_NO_ASK_PASSWORD }, { NULL, 0, NULL, 0 } }; @@ -417,7 +327,7 @@ static int parse_argv(int argc, char *argv[]) { assert(argc >= 0); assert(argv); - while ((c = getopt_long(argc, argv, "hH:P", options, NULL)) >= 0) { + while ((c = getopt_long(argc, argv, "hH:M:", options, NULL)) >= 0) { switch (c) { @@ -430,25 +340,26 @@ static int parse_argv(int argc, char *argv[]) { puts(SYSTEMD_FEATURES); return 0; - case 'P': - arg_transport = TRANSPORT_POLKIT; + case 'H': + arg_transport = BUS_TRANSPORT_REMOTE; + arg_host = optarg; break; - case 'H': - arg_transport = TRANSPORT_SSH; + case 'M': + arg_transport = BUS_TRANSPORT_CONTAINER; arg_host = optarg; break; - case ARG_SET_TRANSIENT: - arg_set_transient = true; + case ARG_TRANSIENT: + arg_transient = true; break; - case ARG_SET_PRETTY: - arg_set_pretty = true; + case ARG_PRETTY: + arg_pretty = true; break; - case ARG_SET_STATIC: - arg_set_static = true; + case ARG_STATIC: + arg_static = true; break; case ARG_NO_ASK_PASSWORD: @@ -464,13 +375,10 @@ static int parse_argv(int argc, char *argv[]) { } } - if (!arg_set_transient && !arg_set_pretty && !arg_set_static) - arg_set_transient = arg_set_pretty = arg_set_static = true; - return 1; } -static int hostnamectl_main(DBusConnection *bus, int argc, char *argv[], DBusError *error) { +static int hostnamectl_main(sd_bus *bus, int argc, char *argv[]) { static const struct { const char* verb; @@ -480,7 +388,7 @@ static int hostnamectl_main(DBusConnection *bus, int argc, char *argv[], DBusErr EQUAL } argc_cmp; const int argc; - int (* const dispatch)(DBusConnection *bus, char **args, unsigned n); + int (* const dispatch)(sd_bus *bus, char **args, unsigned n); } verbs[] = { { "status", LESS, 1, show_status }, { "set-hostname", EQUAL, 2, set_hostname }, @@ -493,7 +401,6 @@ static int hostnamectl_main(DBusConnection *bus, int argc, char *argv[], DBusErr assert(argc >= 0); assert(argv); - assert(error); left = argc - optind; @@ -546,54 +453,29 @@ static int hostnamectl_main(DBusConnection *bus, int argc, char *argv[], DBusErr assert_not_reached("Unknown comparison operator."); } - if (!bus) { - log_error("Failed to get D-Bus connection: %s", error->message); - return -EIO; - } - return verbs[i].dispatch(bus, argv + optind, left); } int main(int argc, char *argv[]) { - int r, retval = EXIT_FAILURE; - DBusConnection *bus = NULL; - DBusError error; - - dbus_error_init(&error); + int r; + _cleanup_bus_unref_ sd_bus *bus = NULL; setlocale(LC_ALL, ""); log_parse_environment(); log_open(); r = parse_argv(argc, argv); - if (r < 0) + if (r <= 0) goto finish; - else if (r == 0) { - retval = EXIT_SUCCESS; + + r = bus_open_transport(arg_transport, arg_host, false, &bus); + if (r < 0) { + log_error("Failed to create bus connection: %s", strerror(-r)); goto finish; } - if (arg_transport == TRANSPORT_NORMAL) - bus = dbus_bus_get_private(DBUS_BUS_SYSTEM, &error); - else if (arg_transport == TRANSPORT_POLKIT) - bus_connect_system_polkit(&bus, &error); - else if (arg_transport == TRANSPORT_SSH) - bus_connect_system_ssh(NULL, arg_host, &bus, &error); - else - assert_not_reached("Uh, invalid transport..."); - - r = hostnamectl_main(bus, argc, argv, &error); - retval = r < 0 ? EXIT_FAILURE : r; + r = hostnamectl_main(bus, argc, argv); finish: - if (bus) { - dbus_connection_flush(bus); - dbus_connection_close(bus); - dbus_connection_unref(bus); - } - - dbus_error_free(&error); - dbus_shutdown(); - - return retval; + return r < 0 ? EXIT_FAILURE : r; }