]> git.ipfire.org Git - thirdparty/systemd.git/commit
sd-event: make sure sd_event_now() cannot fail 843/head
authorLennart Poettering <lennart@poettering.net>
Mon, 3 Aug 2015 15:29:09 +0000 (17:29 +0200)
committerLennart Poettering <lennart@poettering.net>
Mon, 3 Aug 2015 15:34:49 +0000 (17:34 +0200)
commit38a03f06a7393d2721c23f23f0589d2f6d0904af
tree18f293ee4b9269aa085120920e2f36ea4a4fe99b
parent4d2bee0620c04f935f7de0cc10d8d5e5d3e54b6a
sd-event: make sure sd_event_now() cannot fail

Previously, if the event loop never ran before sd_event_now() would
fail. With this change it will instead fall back to invoking now(). This
way, the function cannot fail anymore, except for programming error when
invoking it with wrong parameters.

This takes into account the fact that many callers did not handle the
error condition correctly, and if the callers did, then they kept simply
invoking now() as fall back on their own. Hence let's shorten the code
using this call, and make things more robust, and let's just fall back
to now() internally.

Whether now() is used or the cache timestamp may still be detected via
the return value of sd_event_now(). If > 0 is returned, then the fall
back to now() was used, if == 0 is returned, then the cached value was
returned.

This patch also simplifies many of the invocations of sd_event_now():
the manual fall back to now() can be removed. Also, in cases where the
call is invoked withing void functions we can now protect the invocation
via assert_se(), acknowledging the fact that the call cannot fail
anymore except for programming errors with the parameters.

This change is inspired by #841.
src/libsystemd-network/sd-dhcp-server.c
src/libsystemd-network/sd-dhcp6-client.c
src/libsystemd-network/sd-ipv4ll.c
src/libsystemd-network/sd-pppoe.c
src/libsystemd/sd-event/sd-event.c
src/resolve/resolved-dns-transaction.c
src/udev/udevd.c