From: Yu Watanabe Date: Thu, 26 May 2022 23:16:46 +0000 (+0900) Subject: sd-bus: use UINT32_MAX X-Git-Tag: v252-rc1~891^2~4 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=590a7385621d007211c43dbb15b19279fd523150;p=thirdparty%2Fsystemd.git sd-bus: use UINT32_MAX --- diff --git a/src/libsystemd/sd-bus/bus-dump.c b/src/libsystemd/sd-bus/bus-dump.c index 050eec3a7cd..d67a170a2d9 100644 --- a/src/libsystemd/sd-bus/bus-dump.c +++ b/src/libsystemd/sd-bus/bus-dump.c @@ -79,7 +79,7 @@ _public_ int sd_bus_message_dump(sd_bus_message *m, FILE *f, uint64_t flags) { /* Display synthetic message serial number in a more readable * format than UINT32_MAX */ - if (BUS_MESSAGE_COOKIE(m) == 0xFFFFFFFFULL) + if (BUS_MESSAGE_COOKIE(m) == UINT32_MAX) fprintf(f, " Cookie=-1"); else fprintf(f, " Cookie=%" PRIu64, BUS_MESSAGE_COOKIE(m)); diff --git a/src/libsystemd/sd-bus/bus-message.c b/src/libsystemd/sd-bus/bus-message.c index fbc836aa43e..a157fab6fef 100644 --- a/src/libsystemd/sd-bus/bus-message.c +++ b/src/libsystemd/sd-bus/bus-message.c @@ -406,7 +406,7 @@ static int message_append_reply_cookie(sd_bus_message *m, uint64_t cookie) { return message_append_field_uint64(m, BUS_MESSAGE_HEADER_REPLY_SERIAL, cookie); else { /* 64bit cookies are not supported on dbus1 */ - if (cookie > 0xffffffffUL) + if (cookie > UINT32_MAX) return -EOPNOTSUPP; return message_append_field_uint32(m, BUS_MESSAGE_HEADER_REPLY_SERIAL, (uint32_t) cookie); @@ -2913,7 +2913,7 @@ _public_ int sd_bus_message_seal(sd_bus_message *m, uint64_t cookie, uint64_t ti if (m->poisoned) return -ESTALE; - if (cookie > 0xffffffffULL && + if (cookie > UINT32_MAX && !BUS_MESSAGE_IS_GVARIANT(m)) return -EOPNOTSUPP; diff --git a/src/libsystemd/sd-bus/sd-bus.c b/src/libsystemd/sd-bus/sd-bus.c index c7a58e95cc4..8a3ad7c151c 100644 --- a/src/libsystemd/sd-bus/sd-bus.c +++ b/src/libsystemd/sd-bus/sd-bus.c @@ -1969,7 +1969,7 @@ int bus_seal_synthetic_message(sd_bus *b, sd_bus_message *m) { * pick a fixed, artificial one. We use UINT32_MAX rather * than UINT64_MAX since dbus1 only had 32bit identifiers, * even though kdbus can do 64bit. */ - return sd_bus_message_seal(m, 0xFFFFFFFFULL, 0); + return sd_bus_message_seal(m, UINT32_MAX, 0); } static int bus_write_message(sd_bus *bus, sd_bus_message *m, size_t *idx) {