summaryrefslogtreecommitdiff
path: root/src/timedate
diff options
context:
space:
mode:
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2019-08-04 11:06:33 +0200
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2019-08-04 11:06:33 +0200
commitd9cb4bba2037db946754c414023ede303288cae2 (patch)
tree3f1267c989069de23c40d1565c873fdd7dc2fd12 /src/timedate
parenta4fc96c823043df0dfba6823524a6f9df35bd5ac (diff)
downloadsystemd-d9cb4bba2037db946754c414023ede303288cae2.tar.gz
systemd-d9cb4bba2037db946754c414023ede303288cae2.tar.bz2
systemd-d9cb4bba2037db946754c414023ede303288cae2.zip
timedated: minor code simplifications
Diffstat (limited to 'src/timedate')
-rw-r--r--src/timedate/timedated.c27
1 files changed, 9 insertions, 18 deletions
diff --git a/src/timedate/timedated.c b/src/timedate/timedated.c
index 074ba3c079..706e0ff448 100644
--- a/src/timedate/timedated.c
+++ b/src/timedate/timedated.c
@@ -272,31 +272,25 @@ static int context_read_data(Context *c) {
static int context_write_data_timezone(Context *c) {
_cleanup_free_ char *p = NULL;
- int r = 0;
assert(c);
if (isempty(c->zone)) {
if (unlink("/etc/localtime") < 0 && errno != ENOENT)
- r = -errno;
-
- return r;
+ return -errno;
+ return 0;
}
p = path_join("../usr/share/zoneinfo", c->zone);
if (!p)
return log_oom();
- r = symlink_atomic(p, "/etc/localtime");
- if (r < 0)
- return r;
-
- return 0;
+ return symlink_atomic(p, "/etc/localtime");
}
static int context_write_data_local_rtc(Context *c) {
- int r;
_cleanup_free_ char *s = NULL, *w = NULL;
+ int r;
assert(c);
@@ -543,19 +537,16 @@ static int property_get_rtc_time(
void *userdata,
sd_bus_error *error) {
- struct tm tm;
- usec_t t;
+ struct tm tm = {};
+ usec_t t = 0;
int r;
- zero(tm);
r = clock_get_hwclock(&tm);
- if (r == -EBUSY) {
+ if (r == -EBUSY)
log_warning("/dev/rtc is busy. Is somebody keeping it open continuously? That's not a good idea... Returning a bogus RTC timestamp.");
- t = 0;
- } else if (r == -ENOENT) {
+ else if (r == -ENOENT)
log_debug("/dev/rtc not found.");
- t = 0; /* no RTC found */
- } else if (r < 0)
+ else if (r < 0)
return sd_bus_error_set_errnof(error, r, "Failed to read RTC: %m");
else
t = (usec_t) timegm(&tm) * USEC_PER_SEC;