dasharo/systemd
0
0
Fork 0
mirror of https://github.com/Dasharo/systemd.git synced 2026-03-06 15:02:31 -08:00
Code Issues Packages Projects Releases Wiki Activity

pid1: split out helper func for two similar code paths

Browse Source
This commit is contained in:
Zbigniew Jędrzejewski-Szmek
2019-07-16 12:54:05 +02:00
parent 7842c5f2e2
commit d00a52c737
1 changed files with 40 additions and 46 deletions
Download Patch File Download Diff File Expand all files Collapse all files

86
src/core/dbus-timer.c
View File

@@ -137,6 +137,42 @@ const sd_bus_vtable bus_timer_vtable[] = {
SD_BUS_VTABLE_END SD_BUS_VTABLE_END
}; };
static int timer_add_one_calendar_spec(
Timer *t,
const char *name,
TimerBase base,
UnitWriteFlags flags,
const char *str,
sd_bus_error *error) {
_cleanup_(calendar_spec_freep) CalendarSpec *c = NULL;
int r;
r = calendar_spec_from_string(str, &c);
if (r == -EINVAL)
return sd_bus_error_setf(error, SD_BUS_ERROR_INVALID_ARGS, "Invalid calendar spec");
if (r < 0)
return r;
if (!UNIT_WRITE_FLAGS_NOOP(flags)) {
unit_write_settingf(UNIT(t), flags|UNIT_ESCAPE_SPECIFIERS, name,
"%s=%s", timer_base_to_string(base), str);
TimerValue *v = new(TimerValue, 1);
if (!v)
return -ENOMEM;
*v = (TimerValue) {
.base = base,
.calendar_spec = c,
};
LIST_PREPEND(value, t->values, v);
}
return 1;
};
static int bus_timer_set_transient_property( static int bus_timer_set_transient_property(
Timer *t, Timer *t,
const char *name, const char *name,
@@ -239,36 +275,17 @@ static int bus_timer_set_transient_property(
return r; return r;
while ((r = sd_bus_message_read(message, "(ss)", &base_name, &str)) > 0) { while ((r = sd_bus_message_read(message, "(ss)", &base_name, &str)) > 0) {
_cleanup_(calendar_spec_freep) CalendarSpec *c = NULL;
TimerBase b; TimerBase b;
b = timer_base_from_string(base_name); b = timer_base_from_string(base_name);
if (b != TIMER_CALENDAR) if (b != TIMER_CALENDAR)
return sd_bus_error_setf(error, SD_BUS_ERROR_INVALID_ARGS, "Invalid timer base: %s", base_name); return sd_bus_error_setf(error, SD_BUS_ERROR_INVALID_ARGS,
"Invalid timer base: %s", base_name);
r = calendar_spec_from_string(str, &c); r = timer_add_one_calendar_spec(t, name, b, flags, str, error);
if (r == -EINVAL)
return sd_bus_error_setf(error, SD_BUS_ERROR_INVALID_ARGS, "Invalid calendar spec: %s", str);
if (r < 0) if (r < 0)
return r; return r;
if (!UNIT_WRITE_FLAGS_NOOP(flags)) {
TimerValue *v;
unit_write_settingf(u, flags|UNIT_ESCAPE_SPECIFIERS, name, "%s=%s", base_name, str);
v = new(TimerValue, 1);
if (!v)
return -ENOMEM;
*v = (TimerValue) {
.base = b,
.calendar_spec = TAKE_PTR(c),
};
LIST_PREPEND(value, t->values, v);
}
empty = false; empty = false;
} }
if (r < 0) if (r < 0)
@@ -328,8 +345,6 @@ static int bus_timer_set_transient_property(
} else if (streq(name, "OnCalendar")) { } else if (streq(name, "OnCalendar")) {
TimerValue *v;
_cleanup_(calendar_spec_freep) CalendarSpec *c = NULL;
const char *str; const char *str;
log_notice("Client is using obsolete %s= transient property, please use TimersCalendar= instead.", name); log_notice("Client is using obsolete %s= transient property, please use TimersCalendar= instead.", name);
@@ -338,28 +353,7 @@ static int bus_timer_set_transient_property(
if (r < 0) if (r < 0)
return r; return r;
if (!UNIT_WRITE_FLAGS_NOOP(flags)) { return timer_add_one_calendar_spec(t, name, TIMER_CALENDAR, flags, str, error);
r = calendar_spec_from_string(str, &c);
if (r == -EINVAL)
return sd_bus_error_setf(error, SD_BUS_ERROR_INVALID_ARGS, "Invalid calendar spec");
if (r < 0)
return r;
unit_write_settingf(u, flags|UNIT_ESCAPE_SPECIFIERS, name, "%s=%s", name, str);
v = new(TimerValue, 1);
if (!v)
return -ENOMEM;
*v = (TimerValue) {
.base = TIMER_CALENDAR,
.calendar_spec = TAKE_PTR(c),
};
LIST_PREPEND(value, t->values, v);
}
return 1;
} }
return 0; return 0;