chiark
/
gitweb
/
~ianmdlvl
/
elogind.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
sd-bus: use IN_SET
[elogind.git]
/
src
/
libelogind
/
sd-bus
/
sd-bus.c
diff --git
a/src/libelogind/sd-bus/sd-bus.c
b/src/libelogind/sd-bus/sd-bus.c
index 050d1254dc26fa4e87dd08957199aa7eec50e1e9..6d304719e61f55d82467123aac2e5805d8b14693 100644
(file)
--- a/
src/libelogind/sd-bus/sd-bus.c
+++ b/
src/libelogind/sd-bus/sd-bus.c
@@
-1,5
+1,3
@@
-/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/
-
/***
This file is part of systemd.
/***
This file is part of systemd.
@@
-318,10
+316,7
@@
_public_ int sd_bus_negotiate_creds(sd_bus *bus, int b, uint64_t mask) {
assert_return(!IN_SET(bus->state, BUS_CLOSING, BUS_CLOSED), -EPERM);
assert_return(!bus_pid_changed(bus), -ECHILD);
assert_return(!IN_SET(bus->state, BUS_CLOSING, BUS_CLOSED), -EPERM);
assert_return(!bus_pid_changed(bus), -ECHILD);
- if (b)
- bus->creds_mask |= mask;
- else
- bus->creds_mask &= ~mask;
+ SET_FLAG(bus->creds_mask, mask, b);
/* The well knowns we need unconditionally, so that matches can work */
bus->creds_mask |= SD_BUS_CREDS_WELL_KNOWN_NAMES|SD_BUS_CREDS_UNIQUE_NAME;
/* The well knowns we need unconditionally, so that matches can work */
bus->creds_mask |= SD_BUS_CREDS_WELL_KNOWN_NAMES|SD_BUS_CREDS_UNIQUE_NAME;
@@
-538,7
+533,7
@@
static void skip_address_key(const char **p) {
*p += strcspn(*p, ",");
if (**p == ',')
*p += strcspn(*p, ",");
if (**p == ',')
- (*p)
++;
+ (*p)++;
}
static int parse_unix_address(sd_bus *b, const char **p, char **guid) {
}
static int parse_unix_address(sd_bus *b, const char **p, char **guid) {
@@
-703,7
+698,7
@@
static int parse_exec_address(sd_bus *b, const char **p, char **guid) {
goto fail;
}
goto fail;
}
- (*p)
++;
+ (*p)++;
if (ul >= n_argv) {
if (!GREEDY_REALLOC0(argv, allocated, ul + 2)) {
if (ul >= n_argv) {
if (!GREEDY_REALLOC0(argv, allocated, ul + 2)) {
@@
-1690,7
+1685,7
@@
static int dispatch_wqueue(sd_bus *bus) {
* it got full, then all bets are off
* anyway. */
* it got full, then all bets are off
* anyway. */
- bus->wqueue_size
--;
+ bus->wqueue_size--;
sd_bus_message_unref(bus->wqueue[0]);
memmove(bus->wqueue, bus->wqueue + 1, sizeof(sd_bus_message*) * bus->wqueue_size);
bus->windex = 0;
sd_bus_message_unref(bus->wqueue[0]);
memmove(bus->wqueue, bus->wqueue + 1, sizeof(sd_bus_message*) * bus->wqueue_size);
bus->windex = 0;
@@
-1739,7
+1734,7
@@
static int dispatch_rqueue(sd_bus *bus, bool hint_priority, int64_t priority, sd
/* Dispatch a queued message */
*m = bus->rqueue[0];
/* Dispatch a queued message */
*m = bus->rqueue[0];
- bus->rqueue_size
--;
+ bus->rqueue_size--;
memmove(bus->rqueue, bus->rqueue + 1, sizeof(sd_bus_message*) * bus->rqueue_size);
return 1;
}
memmove(bus->rqueue, bus->rqueue + 1, sizeof(sd_bus_message*) * bus->rqueue_size);
return 1;
}
@@
-1803,7
+1798,7
@@
static int bus_send_internal(sd_bus *bus, sd_bus_message *_m, uint64_t *cookie,
r = bus_write_message(bus, m, hint_sync_call, &idx);
if (r < 0) {
r = bus_write_message(bus, m, hint_sync_call, &idx);
if (r < 0) {
- if (
r == -ENOTCONN || r == -ECONNRESET || r == -EPIPE || r == -ESHUTDOWN
) {
+ if (
IN_SET(r, -ENOTCONN, -ECONNRESET, -EPIPE, -ESHUTDOWN)
) {
bus_enter_closing(bus);
return -ECONNRESET;
}
bus_enter_closing(bus);
return -ECONNRESET;
}
@@
-1831,7
+1826,7
@@
static int bus_send_internal(sd_bus *bus, sd_bus_message *_m, uint64_t *cookie,
if (!GREEDY_REALLOC(bus->wqueue, bus->wqueue_allocated, bus->wqueue_size + 1))
return -ENOMEM;
if (!GREEDY_REALLOC(bus->wqueue, bus->wqueue_allocated, bus->wqueue_size + 1))
return -ENOMEM;
- bus->wqueue[bus->wqueue_size
++] = sd_bus_message_ref(m);
+ bus->wqueue[bus->wqueue_size++] = sd_bus_message_ref(m);
}
finish:
}
finish:
@@
-2110,7
+2105,7
@@
_public_ int sd_bus_call(
r = bus_read_message(bus, false, 0);
if (r < 0) {
r = bus_read_message(bus, false, 0);
if (r < 0) {
- if (
r == -ENOTCONN || r == -ECONNRESET || r == -EPIPE || r == -ESHUTDOWN
) {
+ if (
IN_SET(r, -ENOTCONN, -ECONNRESET, -EPIPE, -ESHUTDOWN)
) {
bus_enter_closing(bus);
r = -ECONNRESET;
}
bus_enter_closing(bus);
r = -ECONNRESET;
}
@@
-2143,7
+2138,7
@@
_public_ int sd_bus_call(
r = dispatch_wqueue(bus);
if (r < 0) {
r = dispatch_wqueue(bus);
if (r < 0) {
- if (
r == -ENOTCONN || r == -ECONNRESET || r == -EPIPE || r == -ESHUTDOWN
) {
+ if (
IN_SET(r, -ENOTCONN, -ECONNRESET, -EPIPE, -ESHUTDOWN)
) {
bus_enter_closing(bus);
r = -ECONNRESET;
}
bus_enter_closing(bus);
r = -ECONNRESET;
}
@@
-2283,7
+2278,7
@@
static int process_timeout(sd_bus *bus) {
slot = container_of(c, sd_bus_slot, reply_callback);
slot = container_of(c, sd_bus_slot, reply_callback);
- bus->iteration_counter
++;
+ bus->iteration_counter++;
bus->current_message = m;
bus->current_slot = sd_bus_slot_ref(slot);
bus->current_message = m;
bus->current_slot = sd_bus_slot_ref(slot);
@@
-2795,7
+2790,7
@@
static int bus_process_internal(sd_bus *bus, bool hint_priority, int64_t priorit
case BUS_OPENING:
r = bus_socket_process_opening(bus);
case BUS_OPENING:
r = bus_socket_process_opening(bus);
- if (
r == -ENOTCONN || r == -ECONNRESET || r == -EPIPE || r == -ESHUTDOWN
) {
+ if (
IN_SET(r, -ENOTCONN, -ECONNRESET, -EPIPE, -ESHUTDOWN)
) {
bus_enter_closing(bus);
r = 1;
} else if (r < 0)
bus_enter_closing(bus);
r = 1;
} else if (r < 0)
@@
-2806,7
+2801,7
@@
static int bus_process_internal(sd_bus *bus, bool hint_priority, int64_t priorit
case BUS_AUTHENTICATING:
r = bus_socket_process_authenticating(bus);
case BUS_AUTHENTICATING:
r = bus_socket_process_authenticating(bus);
- if (
r == -ENOTCONN || r == -ECONNRESET || r == -EPIPE || r == -ESHUTDOWN
) {
+ if (
IN_SET(r, -ENOTCONN, -ECONNRESET, -EPIPE, -ESHUTDOWN)
) {
bus_enter_closing(bus);
r = 1;
} else if (r < 0)
bus_enter_closing(bus);
r = 1;
} else if (r < 0)
@@
-2820,7
+2815,7
@@
static int bus_process_internal(sd_bus *bus, bool hint_priority, int64_t priorit
case BUS_RUNNING:
case BUS_HELLO:
r = process_running(bus, hint_priority, priority, ret);
case BUS_RUNNING:
case BUS_HELLO:
r = process_running(bus, hint_priority, priority, ret);
- if (
r == -ENOTCONN || r == -ECONNRESET || r == -EPIPE || r == -ESHUTDOWN
) {
+ if (
IN_SET(r, -ENOTCONN, -ECONNRESET, -EPIPE, -ESHUTDOWN)
) {
bus_enter_closing(bus);
r = 1;
bus_enter_closing(bus);
r = 1;
@@
-2945,7
+2940,7
@@
_public_ int sd_bus_flush(sd_bus *bus) {
for (;;) {
r = dispatch_wqueue(bus);
if (r < 0) {
for (;;) {
r = dispatch_wqueue(bus);
if (r < 0) {
- if (
r == -ENOTCONN || r == -ECONNRESET || r == -EPIPE || r == -ESHUTDOWN
) {
+ if (
IN_SET(r, -ENOTCONN, -ECONNRESET, -EPIPE, -ESHUTDOWN)
) {
bus_enter_closing(bus);
return -ECONNRESET;
}
bus_enter_closing(bus);
return -ECONNRESET;
}