util: rename write_one_line_file() to write_string_file()
You can write much more than just one line with this call (and we frequently do), so let's correct the naming.
This commit is contained in:
parent
b6e8f1f03d
commit
574d5f2dfc
|
@ -62,7 +62,7 @@ static int delete_rule(const char *rule) {
|
||||||
if (!fn)
|
if (!fn)
|
||||||
return log_oom();
|
return log_oom();
|
||||||
|
|
||||||
return write_one_line_file(fn, "-1");
|
return write_string_file(fn, "-1");
|
||||||
}
|
}
|
||||||
|
|
||||||
static int apply_rule(const char *rule) {
|
static int apply_rule(const char *rule) {
|
||||||
|
@ -70,7 +70,7 @@ static int apply_rule(const char *rule) {
|
||||||
|
|
||||||
delete_rule(rule);
|
delete_rule(rule);
|
||||||
|
|
||||||
r = write_one_line_file("/proc/sys/fs/binfmt_misc/register", rule);
|
r = write_string_file("/proc/sys/fs/binfmt_misc/register", rule);
|
||||||
if (r < 0) {
|
if (r < 0) {
|
||||||
log_error("Failed to add binary format: %s", strerror(-r));
|
log_error("Failed to add binary format: %s", strerror(-r));
|
||||||
return r;
|
return r;
|
||||||
|
@ -197,7 +197,7 @@ int main(int argc, char *argv[]) {
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Flush out all rules */
|
/* Flush out all rules */
|
||||||
write_one_line_file("/proc/sys/fs/binfmt_misc/status", "-1");
|
write_string_file("/proc/sys/fs/binfmt_misc/status", "-1");
|
||||||
|
|
||||||
STRV_FOREACH(f, files) {
|
STRV_FOREACH(f, files) {
|
||||||
k = apply_file(*f, true);
|
k = apply_file(*f, true);
|
||||||
|
|
|
@ -44,7 +44,7 @@ int cgroup_attribute_apply(CGroupAttribute *a, CGroupBonding *b) {
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
r = write_one_line_file(path, v ? v : a->value);
|
r = write_string_file(path, v ? v : a->value);
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
log_warning("Failed to write '%s' to %s: %s", v ? v : a->value, path, strerror(-r));
|
log_warning("Failed to write '%s' to %s: %s", v ? v : a->value, path, strerror(-r));
|
||||||
|
|
||||||
|
|
|
@ -1195,7 +1195,7 @@ int exec_spawn(ExecCommand *command,
|
||||||
snprintf(t, sizeof(t), "%i", context->oom_score_adjust);
|
snprintf(t, sizeof(t), "%i", context->oom_score_adjust);
|
||||||
char_array_0(t);
|
char_array_0(t);
|
||||||
|
|
||||||
if (write_one_line_file("/proc/self/oom_score_adj", t) < 0) {
|
if (write_string_file("/proc/self/oom_score_adj", t) < 0) {
|
||||||
err = -errno;
|
err = -errno;
|
||||||
r = EXIT_OOM_ADJUST;
|
r = EXIT_OOM_ADJUST;
|
||||||
goto fail_child;
|
goto fail_child;
|
||||||
|
|
|
@ -216,7 +216,7 @@ int machine_id_setup(void) {
|
||||||
* /run/machine-id as a replacement */
|
* /run/machine-id as a replacement */
|
||||||
|
|
||||||
m = umask(0022);
|
m = umask(0022);
|
||||||
r = write_one_line_file("/run/machine-id", id);
|
r = write_string_file("/run/machine-id", id);
|
||||||
umask(m);
|
umask(m);
|
||||||
|
|
||||||
if (r < 0) {
|
if (r < 0) {
|
||||||
|
|
|
@ -2910,7 +2910,7 @@ int unit_write_drop_in(Unit *u, bool runtime, const char *name, const char *data
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
mkdir_p(p, 0755);
|
mkdir_p(p, 0755);
|
||||||
return write_one_line_file_atomic_label(q, data);
|
return write_string_file_atomic_label(q, data);
|
||||||
}
|
}
|
||||||
|
|
||||||
int unit_remove_drop_in(Unit *u, bool runtime, const char *name) {
|
int unit_remove_drop_in(Unit *u, bool runtime, const char *name) {
|
||||||
|
|
|
@ -289,7 +289,7 @@ static int write_data_static_hostname(void) {
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
return write_one_line_file_atomic_label("/etc/hostname", data[PROP_STATIC_HOSTNAME]);
|
return write_string_file_atomic_label("/etc/hostname", data[PROP_STATIC_HOSTNAME]);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int write_data_other(void) {
|
static int write_data_other(void) {
|
||||||
|
|
|
@ -886,7 +886,7 @@ static int trigger_device(Manager *m, struct udev_device *d) {
|
||||||
goto finish;
|
goto finish;
|
||||||
}
|
}
|
||||||
|
|
||||||
write_one_line_file(t, "change");
|
write_string_file(t, "change");
|
||||||
free(t);
|
free(t);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -936,7 +936,7 @@ static int attach_device(Manager *m, const char *seat, const char *sysfs) {
|
||||||
|
|
||||||
mkdir_p_label("/etc/udev/rules.d", 0755);
|
mkdir_p_label("/etc/udev/rules.d", 0755);
|
||||||
label_init("/etc");
|
label_init("/etc");
|
||||||
r = write_one_line_file_atomic_label(file, rule);
|
r = write_string_file_atomic_label(file, rule);
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
goto finish;
|
goto finish;
|
||||||
|
|
||||||
|
|
|
@ -70,7 +70,7 @@ int main(int argc, char*argv[]) {
|
||||||
int r, q;
|
int r, q;
|
||||||
char *cgroup_user_tree = NULL;
|
char *cgroup_user_tree = NULL;
|
||||||
|
|
||||||
r = write_one_line_file_atomic("/run/nologin", "System is going down.");
|
r = write_string_file_atomic("/run/nologin", "System is going down.");
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
log_error("Failed to create /run/nologin: %s", strerror(-r));
|
log_error("Failed to create /run/nologin: %s", strerror(-r));
|
||||||
|
|
||||||
|
|
|
@ -528,7 +528,7 @@ static int setup_boot_id(const char *dest) {
|
||||||
SD_ID128_FORMAT_VAL(rnd));
|
SD_ID128_FORMAT_VAL(rnd));
|
||||||
char_array_0(as_uuid);
|
char_array_0(as_uuid);
|
||||||
|
|
||||||
r = write_one_line_file(from, as_uuid);
|
r = write_string_file(from, as_uuid);
|
||||||
if (r < 0) {
|
if (r < 0) {
|
||||||
log_error("Failed to write boot id: %s", strerror(-r));
|
log_error("Failed to write boot id: %s", strerror(-r));
|
||||||
return r;
|
return r;
|
||||||
|
|
|
@ -316,7 +316,7 @@ int block_bump_request_nr(const char *p) {
|
||||||
goto finish;
|
goto finish;
|
||||||
}
|
}
|
||||||
|
|
||||||
r = write_one_line_file(ap, line);
|
r = write_string_file(ap, line);
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
goto finish;
|
goto finish;
|
||||||
|
|
||||||
|
@ -399,7 +399,7 @@ int block_set_readahead(const char *p, uint64_t bytes) {
|
||||||
goto finish;
|
goto finish;
|
||||||
}
|
}
|
||||||
|
|
||||||
r = write_one_line_file(ap, line);
|
r = write_string_file(ap, line);
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
goto finish;
|
goto finish;
|
||||||
|
|
||||||
|
|
|
@ -204,7 +204,7 @@ static int drop_from_file(const char *fn, uint64_t drop) {
|
||||||
if (asprintf(&p, "%u %u", lo, hi) < 0)
|
if (asprintf(&p, "%u %u", lo, hi) < 0)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
r = write_one_line_file(fn, p);
|
r = write_string_file(fn, p);
|
||||||
free(p);
|
free(p);
|
||||||
|
|
||||||
return r;
|
return r;
|
||||||
|
|
|
@ -677,9 +677,9 @@ int cg_delete(const char *controller, const char *path) {
|
||||||
}
|
}
|
||||||
|
|
||||||
int cg_attach(const char *controller, const char *path, pid_t pid) {
|
int cg_attach(const char *controller, const char *path, pid_t pid) {
|
||||||
char *fs;
|
_cleanup_free_ char *fs = NULL;
|
||||||
|
char c[DECIMAL_STR_MAX(pid_t) + 2];
|
||||||
int r;
|
int r;
|
||||||
char c[32];
|
|
||||||
|
|
||||||
assert(controller);
|
assert(controller);
|
||||||
assert(path);
|
assert(path);
|
||||||
|
@ -693,16 +693,12 @@ int cg_attach(const char *controller, const char *path, pid_t pid) {
|
||||||
pid = getpid();
|
pid = getpid();
|
||||||
|
|
||||||
snprintf(c, sizeof(c), "%lu\n", (unsigned long) pid);
|
snprintf(c, sizeof(c), "%lu\n", (unsigned long) pid);
|
||||||
char_array_0(c);
|
|
||||||
|
|
||||||
r = write_one_line_file(fs, c);
|
return write_string_file(fs, c);
|
||||||
free(fs);
|
|
||||||
|
|
||||||
return r;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
int cg_set_group_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid) {
|
int cg_set_group_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid) {
|
||||||
char *fs;
|
_cleanup_free_ char *fs = NULL;
|
||||||
int r;
|
int r;
|
||||||
|
|
||||||
assert(controller);
|
assert(controller);
|
||||||
|
@ -715,10 +711,7 @@ int cg_set_group_access(const char *controller, const char *path, mode_t mode, u
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
r = chmod_and_chown(fs, mode, uid, gid);
|
return chmod_and_chown(fs, mode, uid, gid);
|
||||||
free(fs);
|
|
||||||
|
|
||||||
return r;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
int cg_set_task_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid, int sticky) {
|
int cg_set_task_access(const char *controller, const char *path, mode_t mode, uid_t uid, gid_t gid, int sticky) {
|
||||||
|
@ -857,7 +850,8 @@ int cg_install_release_agent(const char *controller, const char *agent) {
|
||||||
goto finish;
|
goto finish;
|
||||||
}
|
}
|
||||||
|
|
||||||
if ((r = write_one_line_file(fs, line)) < 0)
|
r = write_string_file(fs, line);
|
||||||
|
if (r < 0)
|
||||||
goto finish;
|
goto finish;
|
||||||
|
|
||||||
} else if (!streq(sc, agent)) {
|
} else if (!streq(sc, agent)) {
|
||||||
|
@ -878,7 +872,7 @@ int cg_install_release_agent(const char *controller, const char *agent) {
|
||||||
sc = strstrip(contents);
|
sc = strstrip(contents);
|
||||||
|
|
||||||
if (streq(sc, "0")) {
|
if (streq(sc, "0")) {
|
||||||
if ((r = write_one_line_file(fs, "1\n")) < 0)
|
if ((r = write_string_file(fs, "1\n")) < 0)
|
||||||
goto finish;
|
goto finish;
|
||||||
|
|
||||||
r = 1;
|
r = 1;
|
||||||
|
|
|
@ -26,14 +26,14 @@
|
||||||
#include "fileio-label.h"
|
#include "fileio-label.h"
|
||||||
#include "label.h"
|
#include "label.h"
|
||||||
|
|
||||||
int write_one_line_file_atomic_label(const char *fn, const char *line) {
|
int write_string_file_atomic_label(const char *fn, const char *line) {
|
||||||
int r;
|
int r;
|
||||||
|
|
||||||
r = label_context_set(fn, S_IFREG);
|
r = label_context_set(fn, S_IFREG);
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
return r;
|
return r;
|
||||||
|
|
||||||
write_one_line_file_atomic(fn, line);
|
write_string_file_atomic(fn, line);
|
||||||
|
|
||||||
label_context_clear();
|
label_context_clear();
|
||||||
|
|
||||||
|
|
|
@ -25,5 +25,5 @@
|
||||||
#include <stdio.h>
|
#include <stdio.h>
|
||||||
#include "fileio.h"
|
#include "fileio.h"
|
||||||
|
|
||||||
int write_one_line_file_atomic_label(const char *fn, const char *line);
|
int write_string_file_atomic_label(const char *fn, const char *line);
|
||||||
int write_env_file_label(const char *fname, char **l);
|
int write_env_file_label(const char *fname, char **l);
|
||||||
|
|
|
@ -24,7 +24,7 @@
|
||||||
#include "util.h"
|
#include "util.h"
|
||||||
#include "strv.h"
|
#include "strv.h"
|
||||||
|
|
||||||
int write_one_line_file(const char *fn, const char *line) {
|
int write_string_file(const char *fn, const char *line) {
|
||||||
_cleanup_fclose_ FILE *f = NULL;
|
_cleanup_fclose_ FILE *f = NULL;
|
||||||
|
|
||||||
assert(fn);
|
assert(fn);
|
||||||
|
@ -49,7 +49,7 @@ int write_one_line_file(const char *fn, const char *line) {
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
int write_one_line_file_atomic(const char *fn, const char *line) {
|
int write_string_file_atomic(const char *fn, const char *line) {
|
||||||
_cleanup_fclose_ FILE *f = NULL;
|
_cleanup_fclose_ FILE *f = NULL;
|
||||||
_cleanup_free_ char *p = NULL;
|
_cleanup_free_ char *p = NULL;
|
||||||
int r;
|
int r;
|
||||||
|
|
|
@ -23,8 +23,8 @@
|
||||||
#include <stddef.h>
|
#include <stddef.h>
|
||||||
#include "macro.h"
|
#include "macro.h"
|
||||||
|
|
||||||
int write_one_line_file(const char *fn, const char *line);
|
int write_string_file(const char *fn, const char *line);
|
||||||
int write_one_line_file_atomic(const char *fn, const char *line);
|
int write_string_file_atomic(const char *fn, const char *line);
|
||||||
int read_one_line_file(const char *fn, char **line);
|
int read_one_line_file(const char *fn, char **line);
|
||||||
int read_full_file(const char *fn, char **contents, size_t *size);
|
int read_full_file(const char *fn, char **contents, size_t *size);
|
||||||
|
|
||||||
|
|
|
@ -421,7 +421,7 @@ int main(int argc, char *argv[]) {
|
||||||
|
|
||||||
log_info("Creating /run/nologin, blocking further logins...");
|
log_info("Creating /run/nologin, blocking further logins...");
|
||||||
|
|
||||||
e = write_one_line_file_atomic("/run/nologin", "System is going down.");
|
e = write_string_file_atomic("/run/nologin", "System is going down.");
|
||||||
if (e < 0)
|
if (e < 0)
|
||||||
log_error("Failed to create /run/nologin: %s", strerror(-e));
|
log_error("Failed to create /run/nologin: %s", strerror(-e));
|
||||||
else
|
else
|
||||||
|
|
|
@ -57,12 +57,12 @@ int main(int argc, char *argv[]) {
|
||||||
|
|
||||||
/* Configure the hibernation mode */
|
/* Configure the hibernation mode */
|
||||||
if (streq(argv[1], "hibernate")) {
|
if (streq(argv[1], "hibernate")) {
|
||||||
if (write_one_line_file("/sys/power/disk", "platform") < 0)
|
if (write_string_file("/sys/power/disk", "platform") < 0)
|
||||||
write_one_line_file("/sys/power/disk", "shutdown");
|
write_string_file("/sys/power/disk", "shutdown");
|
||||||
} else if (streq(argv[1], "hybrid-sleep")) {
|
} else if (streq(argv[1], "hybrid-sleep")) {
|
||||||
if (write_one_line_file("/sys/power/disk", "suspend") < 0)
|
if (write_string_file("/sys/power/disk", "suspend") < 0)
|
||||||
if (write_one_line_file("/sys/power/disk", "platform") < 0)
|
if (write_string_file("/sys/power/disk", "platform") < 0)
|
||||||
write_one_line_file("/sys/power/disk", "shutdown");
|
write_string_file("/sys/power/disk", "shutdown");
|
||||||
}
|
}
|
||||||
|
|
||||||
f = fopen("/sys/power/state", "we");
|
f = fopen("/sys/power/state", "we");
|
||||||
|
|
|
@ -88,7 +88,7 @@ static int apply_sysctl(const char *property, const char *value) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
k = write_one_line_file(p, value);
|
k = write_string_file(p, value);
|
||||||
if (k < 0) {
|
if (k < 0) {
|
||||||
log_full(k == -ENOENT ? LOG_DEBUG : LOG_WARNING,
|
log_full(k == -ENOENT ? LOG_DEBUG : LOG_WARNING,
|
||||||
"Failed to write '%s' to '%s': %s", value, p, strerror(-k));
|
"Failed to write '%s' to '%s': %s", value, p, strerror(-k));
|
||||||
|
|
|
@ -275,7 +275,7 @@ static int write_data_local_rtc(void) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
label_init("/etc");
|
label_init("/etc");
|
||||||
r = write_one_line_file_atomic_label("/etc/adjtime", w);
|
r = write_string_file_atomic_label("/etc/adjtime", w);
|
||||||
free(w);
|
free(w);
|
||||||
|
|
||||||
return r;
|
return r;
|
||||||
|
|
|
@ -267,7 +267,7 @@ static void worker_new(struct event *event)
|
||||||
prctl(PR_SET_PDEATHSIG, SIGTERM);
|
prctl(PR_SET_PDEATHSIG, SIGTERM);
|
||||||
|
|
||||||
/* reset OOM score, we only protect the main daemon */
|
/* reset OOM score, we only protect the main daemon */
|
||||||
write_one_line_file("/proc/self/oom_score_adj", "0");
|
write_string_file("/proc/self/oom_score_adj", "0");
|
||||||
|
|
||||||
for (;;) {
|
for (;;) {
|
||||||
struct udev_event *udev_event;
|
struct udev_event *udev_event;
|
||||||
|
@ -1168,7 +1168,7 @@ int main(int argc, char *argv[])
|
||||||
|
|
||||||
setsid();
|
setsid();
|
||||||
|
|
||||||
write_one_line_file("/proc/self/oom_score_adj", "-1000");
|
write_string_file("/proc/self/oom_score_adj", "-1000");
|
||||||
} else {
|
} else {
|
||||||
sd_notify(1, "READY=1");
|
sd_notify(1, "READY=1");
|
||||||
}
|
}
|
||||||
|
|
|
@ -57,7 +57,7 @@ static int disable_utf8(int fd) {
|
||||||
if (loop_write(fd, "\033%@", 3, false) < 0)
|
if (loop_write(fd, "\033%@", 3, false) < 0)
|
||||||
r = -errno;
|
r = -errno;
|
||||||
|
|
||||||
k = write_one_line_file("/sys/module/vt/parameters/default_utf8", "0");
|
k = write_string_file("/sys/module/vt/parameters/default_utf8", "0");
|
||||||
if (k < 0)
|
if (k < 0)
|
||||||
r = k;
|
r = k;
|
||||||
|
|
||||||
|
@ -89,7 +89,7 @@ static int enable_utf8(int fd) {
|
||||||
if (loop_write(fd, "\033%G", 3, false) < 0)
|
if (loop_write(fd, "\033%G", 3, false) < 0)
|
||||||
r = -errno;
|
r = -errno;
|
||||||
|
|
||||||
k = write_one_line_file("/sys/module/vt/parameters/default_utf8", "1");
|
k = write_string_file("/sys/module/vt/parameters/default_utf8", "1");
|
||||||
if (k < 0)
|
if (k < 0)
|
||||||
r = k;
|
r = k;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue