diff options
| author | Lennart Poettering <lennart@poettering.net> | 2022-02-03 17:41:04 +0100 |
|---|---|---|
| committer | Luca Boccassi <luca.boccassi@gmail.com> | 2022-02-03 19:52:18 +0000 |
| commit | ff28d259c0b0630841878d503e31cc9480c50f5d (patch) | |
| tree | a65f28310723de5f1e41476929dc379916cce495 /src/hostname | |
| parent | 79decacff0029e92d71f0871defee5989fb50674 (diff) | |
| download | systemd-ff28d259c0b0630841878d503e31cc9480c50f5d.tar.gz | |
hostnamed: drop "iteractive" parameter from GetHardwareSerial()
Since a long time the D-Bus spec knows a special bit in its message
header for indicating that "interactive" authentication is OK. The
original hostnamed API is before that was added hence most functions
expose that boolean as explicit argument.
For new added functions let's get rid of it, the message flag is good
enough and replaces it with complete functionality.
No new APIs should carry the "interactive" boolean flag explicitly as
argument anymore.
Follow-up for: 9697662915e47a4797b05003cb1970fe2b01e530
Diffstat (limited to 'src/hostname')
| -rw-r--r-- | src/hostname/hostnamed.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/src/hostname/hostnamed.c b/src/hostname/hostnamed.c index 4637ac8817..a60e0f8295 100644 --- a/src/hostname/hostnamed.c +++ b/src/hostname/hostnamed.c @@ -1082,21 +1082,17 @@ static int method_get_hardware_serial(sd_bus_message *m, void *userdata, sd_bus_ _cleanup_(sd_bus_message_unrefp) sd_bus_message *reply = NULL; _cleanup_free_ char *serial = NULL; Context *c = userdata; - int interactive, r; + int r; assert(m); assert(c); - r = sd_bus_message_read(m, "b", &interactive); - if (r < 0) - return r; - r = bus_verify_polkit_async( m, CAP_SYS_ADMIN, "org.freedesktop.hostname1.get-hardware-serial", NULL, - interactive, + false, UID_INVALID, &c->polkit_registry, error); @@ -1304,8 +1300,7 @@ static const sd_bus_vtable hostname_vtable[] = { method_get_product_uuid, SD_BUS_VTABLE_UNPRIVILEGED), SD_BUS_METHOD_WITH_NAMES("GetHardwareSerial", - "b", - SD_BUS_PARAM(interactive), + NULL,, "s", SD_BUS_PARAM(serial), method_get_hardware_serial, |