21770421410

Committed 06 Feb 2026 11:15PM UTC coverage: 72.736% (+0.1%) from 72.601%

Build # 21770421410

Build Type

push

github

Committed by

bluca

Commit Message

daemon-util: downgrade log level on ECONNREFUSED and friends

This partially reverts 36c557f7d, which
introduced notify_remove_fd() that logs in LOG_DEBUG. However,
notify_remove_fd_warn() is still called other library functions, e.g.
notify_push_fd(), and produces warning message about the failure in
removing fd from fdstore on shutdown.

During shutdown process, we get the following logs:
```
systemd-udevd[370]: Failed to send notify message to '/run/systemd/notify': Connection refused
systemd-udevd[370]: Failed to remove file descriptor "config-serialization" from the store, ignoring: Connection refused
systemd-udevd[370]: Failed to send notify message to '/run/systemd/notify': Connection refused
systemd-udevd[370]: Failed to push serialization fd to service manager: Connection refused
```
Here, the 1st, 3rd, and 4th messages are in LOG_DEBUG, but the 2nd one
was in LOG_WARNING before this commit, and this makes it also in LOG_DEBUG.

Follow-up for 472404aca.

Run Details

2 of 2 new or added lines in 2 files covered. (100.0%)

430 existing lines in 30 files now uncovered.

311628 of 428439 relevant lines covered (72.74%)

1148312.85 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

88.46

/src/shared/serialize.c

/* SPDX-License-Identifier: LGPL-2.1-or-later */

#include <fcntl.h>
#include <unistd.h>

#include "sd-id128.h"

#include "alloc-util.h"
#include "env-util.h"
#include "escape.h"
#include "extract-word.h"
#include "fd-util.h"
#include "fdset.h"
#include "fileio.h"
#include "format-util.h"
#include "hexdecoct.h"
#include "image-policy.h"
#include "log.h"
#include "memfd-util.h"
#include "parse-util.h"
#include "pidref.h"
#include "ratelimit.h"
#include "serialize.h"
#include "set.h"
#include "string-util.h"
#include "strv.h"
#include "time-util.h"

int serialize_item(FILE *f, const char *key, const char *value) {
        assert(f);
        assert(key);

        if (!value)
                return 0;

        /* Make sure that anything we serialize we can also read back again with read_line() with a maximum line size
         * of LONG_LINE_MAX. This is a safety net only. All code calling us should filter this out earlier anyway. */
        if (strlen(key) + 1 + strlen(value) + 1 > LONG_LINE_MAX)
                return log_warning_errno(SYNTHETIC_ERRNO(EINVAL), "Attempted to serialize overly long item '%s', refusing.", key);

        fputs(key, f);
        fputc('=', f);
        fputs(value, f);
        fputc('\n', f);

        return 1;
}

int serialize_item_escaped(FILE *f, const char *key, const char *value) {
        _cleanup_free_ char *c = NULL;

        assert(f);
        assert(key);

        if (!value)
                return 0;

        c = xescape(value, " ");
        if (!c)
                return log_oom();

        return serialize_item(f, key, c);
}

int serialize_item_format(FILE *f, const char *key, const char *format, ...) {
        _cleanup_free_ char *allocated = NULL;
        char buf[256]; /* Something reasonably short that fits nicely on any stack (i.e. is considerably less
                        * than LONG_LINE_MAX (1MiB!) */
        const char *b;
        va_list ap;
        int k;

        assert(f);
        assert(key);
        assert(format);

        /* First, let's try to format this into a stack buffer */
        va_start(ap, format);
        k = vsnprintf(buf, sizeof(buf), format, ap);
        va_end(ap);

        if (k < 0)
                return log_warning_errno(errno, "Failed to serialize item '%s', ignoring: %m", key);
        if (strlen(key) + 1 + k + 1 > LONG_LINE_MAX) /* See above */
                return log_warning_errno(SYNTHETIC_ERRNO(EINVAL), "Attempted to serialize overly long item '%s', refusing.", key);

        if ((size_t) k < sizeof(buf))
                b = buf; /* Yay, it fit! */
        else {
                /* So the string didn't fit in the short buffer above, but was not above our total limit,
                 * hence let's format it via dynamic memory */

                va_start(ap, format);
                k = vasprintf(&allocated, format, ap);
                va_end(ap);

                if (k < 0)
                        return log_warning_errno(errno, "Failed to serialize item '%s', ignoring: %m", key);

                b = allocated;
        }

        fputs(key, f);
        fputc('=', f);
        fputs(b, f);
        fputc('\n', f);

        return 1;
}

int serialize_fd(FILE *f, FDSet *fds, const char *key, int fd) {
        int copy;

        assert(f);
        assert(fds);
        assert(key);

        if (fd < 0)
                return 0;

        copy = fdset_put_dup(fds, fd);
        if (copy < 0)
                return log_error_errno(copy, "Failed to add file descriptor to serialization set: %m");

        return serialize_item_format(f, key, "%i", copy);
}

int serialize_fd_many(FILE *f, FDSet *fds, const char *key, const int fd_array[], size_t n_fd_array) {
        _cleanup_free_ char *t = NULL;

        assert(f);

        if (n_fd_array == 0)
                return 0;

        assert(fd_array);

        for (size_t i = 0; i < n_fd_array; i++) {
                int copy;

                if (fd_array[i] < 0)
                        return -EBADF;

                copy = fdset_put_dup(fds, fd_array[i]);
                if (copy < 0)
                        return log_error_errno(copy, "Failed to add file descriptor to serialization set: %m");

                if (strextendf_with_separator(&t, " ", "%i", copy) < 0)
                        return log_oom();
        }

        return serialize_item(f, key, t);
}

int serialize_usec(FILE *f, const char *key, usec_t usec) {
        assert(f);
        assert(key);

        if (usec == USEC_INFINITY)
                return 0;

        return serialize_item_format(f, key, USEC_FMT, usec);
}

int serialize_dual_timestamp(FILE *f, const char *key, const dual_timestamp *t) {
        assert(f);
        assert(key);
        assert(t);

        if (!dual_timestamp_is_set(t))
                return 0;

        return serialize_item_format(f, key, USEC_FMT " " USEC_FMT, t->realtime, t->monotonic);
}

int serialize_strv(FILE *f, const char *key, char * const *l) {
        int ret = 0, r;

        /* Returns the first error, or positive if anything was serialized, 0 otherwise. */

        assert(f);
        assert(key);

        STRV_FOREACH(i, l) {
                r = serialize_item_escaped(f, key, *i);
                if ((ret >= 0 && r < 0) ||
                    (ret == 0 && r > 0))
                        ret = r;
        }

        return ret;
}

int serialize_id128(FILE *f, const char *key, sd_id128_t id) {
        assert(f);
        assert(key);

        if (sd_id128_is_null(id))
                return 0;

        return serialize_item_format(f, key, SD_ID128_FORMAT_STR, SD_ID128_FORMAT_VAL(id));
}

int serialize_pidref(FILE *f, FDSet *fds, const char *key, PidRef *pidref) {
        int r;

        assert(f);
        assert(fds);

        if (!pidref_is_set(pidref))
                return 0;

        /* We always serialize the pid separately, to keep downgrades mostly working (older versions will
         * deserialize the pid and silently fail to deserialize the pidfd). If we also have a pidfd, we
         * serialize both the pid and pidfd, so that we can construct the exact same pidref after
         * deserialization (this doesn't work with only the pidfd, as we can't retrieve the original pid
         * from the pidfd anymore if the process is reaped). */

        if (pidref->fd >= 0) {
                int copy = fdset_put_dup(fds, pidref->fd);
                if (copy < 0)
                        return log_error_errno(copy, "Failed to add file descriptor to serialization set: %m");

                r = serialize_item_format(f, key, "@%i:" PID_FMT, copy, pidref->pid);
                if (r < 0)
                        return r;
        }

        return serialize_item_format(f, key, PID_FMT, pidref->pid);
}

int serialize_ratelimit(FILE *f, const char *key, const RateLimit *rl) {
        assert(rl);

        return serialize_item_format(f, key,
                                     USEC_FMT " " USEC_FMT " %u %u",
                                     rl->begin,
                                     rl->interval,
                                     rl->num,
                                     rl->burst);
}

int serialize_item_hexmem(FILE *f, const char *key, const void *p, size_t l) {
        _cleanup_free_ char *encoded = NULL;
        int r;

        assert(f);
        assert(key);

        if (!p && l > 0)
                return -EINVAL;

        if (l == 0)
                return 0;

        encoded = hexmem(p, l);
        if (!encoded)
                return log_oom_debug();

        r = serialize_item(f, key, encoded);
        if (r < 0)
                return r;

        return 1;
}

int serialize_item_base64mem(FILE *f, const char *key, const void *p, size_t l) {
        _cleanup_free_ char *encoded = NULL;
        ssize_t len;
        int r;

        assert(f);
        assert(key);

        if (!p && l > 0)
                return -EINVAL;

        if (l == 0)
                return 0;

        len = base64mem(p, l, &encoded);
        if (len <= 0)
                return log_oom_debug();

        r = serialize_item(f, key, encoded);
        if (r < 0)
                return r;

        return 1;
}

int serialize_string_set(FILE *f, const char *key, const Set *s) {
        int r;

        assert(f);
        assert(key);

        if (set_isempty(s))
                return 0;

        /* Serialize as individual items, as each element might contain separators and escapes */

        const char *e;
        SET_FOREACH(e, s) {
                r = serialize_item(f, key, e);
                if (r < 0)
                        return r;
        }

        return 1;
}

int serialize_image_policy(FILE *f, const char *key, const ImagePolicy *p) {
        _cleanup_free_ char *policy = NULL;
        int r;

        assert(f);
        assert(key);

        if (!p)
                return 0;

        r = image_policy_to_string(p, /* simplify= */ false, &policy);
        if (r < 0)
                return r;

        r = serialize_item(f, key, policy);
        if (r < 0)
                return r;

        return 1;
}

int serialize_bool(FILE *f, const char *key, bool b) {
        return serialize_item(f, key, yes_no(b));
}

int serialize_bool_elide(FILE *f, const char *key, bool b) {
        return b ? serialize_item(f, key, yes_no(b)) : 0;
}

int deserialize_read_line(FILE *f, char **ret) {
        _cleanup_free_ char *line = NULL;
        int r;

        assert(f);
        assert(ret);

        r = read_stripped_line(f, LONG_LINE_MAX, &line);
        if (r < 0)
                return log_error_errno(r, "Failed to read serialization line: %m");
        if (r == 0) { /* eof */
                *ret = NULL;
                return 0;
        }

        if (isempty(line)) { /* End marker */
                *ret = NULL;
                return 0;
        }

        *ret = TAKE_PTR(line);
        return 1;
}

int deserialize_fd(FDSet *fds, const char *value) {
        _cleanup_close_ int our_fd = -EBADF;
        int parsed_fd;

        assert(value);

        parsed_fd = parse_fd(value);
        if (parsed_fd < 0)
                return log_debug_errno(parsed_fd, "Failed to parse file descriptor serialization: %s", value);

        our_fd = fdset_remove(fds, parsed_fd); /* Take possession of the fd */
        if (our_fd < 0)
                return log_debug_errno(our_fd, "Failed to acquire fd from serialization fds: %m");

        return TAKE_FD(our_fd);
}

int deserialize_fd_many(FDSet *fds, const char *value, size_t n, int *ret) {
        int r, *fd_array = NULL;
        size_t m = 0;

        assert(value);

        fd_array = new(int, n);
        if (!fd_array)
                return -ENOMEM;

        CLEANUP_ARRAY(fd_array, m, close_many_and_free);

        for (;;) {
                _cleanup_free_ char *w = NULL;
                int fd;

                r = extract_first_word(&value, &w, NULL, 0);
                if (r < 0)
                        return r;
                if (r == 0) {
                        if (m < n) /* Too few */
                                return -EINVAL;

                        break;
                }

                if (m >= n) /* Too many */
                        return -EINVAL;

                fd = deserialize_fd(fds, w);
                if (fd < 0)
                        return fd;

                fd_array[m++] = fd;
        }

        memcpy(ret, fd_array, m * sizeof(int));
        fd_array = mfree(fd_array);

        return 0;
}

int deserialize_strv(const char *value, char ***l) {
        ssize_t unescaped_len;
        char *unescaped;

        assert(l);
        assert(value);

        unescaped_len = cunescape(value, 0, &unescaped);
        if (unescaped_len < 0)
                return unescaped_len;

        return strv_consume(l, unescaped);
}

int deserialize_usec(const char *value, usec_t *ret) {
        int r;

        assert(value);
        assert(ret);

        r = safe_atou64(value, ret);
        if (r < 0)
                return log_debug_errno(r, "Failed to parse usec value \"%s\": %m", value);

        return 0;
}

int deserialize_dual_timestamp(const char *value, dual_timestamp *ret) {
        uint64_t a, b;
        int r, pos;

        assert(value);
        assert(ret);

        pos = strspn(value, WHITESPACE);
        if (value[pos] == '-')
                return -EINVAL;
        pos += strspn(value + pos, DIGITS);
        pos += strspn(value + pos, WHITESPACE);
        if (value[pos] == '-')
                return -EINVAL;

        r = sscanf(value, "%" PRIu64 "%" PRIu64 "%n", &a, &b, &pos);
        if (r != 2)
                return log_debug_errno(SYNTHETIC_ERRNO(EINVAL),
                                       "Failed to parse dual timestamp value \"%s\".",
                                       value);

        if (value[pos] != '\0')
                /* trailing garbage */
                return -EINVAL;

        *ret = (dual_timestamp) {
                .realtime = a,
                .monotonic = b,
        };

        return 0;
}

int deserialize_environment(const char *value, char ***list) {
        _cleanup_free_ char *unescaped = NULL;
        ssize_t l;
        int r;

        assert(value);
        assert(list);

        /* Changes the *environment strv inline. */

        l = cunescape(value, 0, &unescaped);
        if (l < 0)
                return log_error_errno(l, "Failed to unescape: %m");

        r = strv_env_replace_consume(list, TAKE_PTR(unescaped));
        if (r < 0)
                return log_error_errno(r, "Failed to append environment variable: %m");

        return 0;
}

int deserialize_pidref(FDSet *fds, const char *value, PidRef *ret) {
        const char *e;
        int r;

        assert(value);
        assert(ret);

        e = startswith(value, "@");
        if (e) {
                _cleanup_free_ char *fdstr = NULL, *pidstr = NULL;
                _cleanup_close_ int fd = -EBADF;

                r = extract_many_words(&e, ":", /* flags= */ 0, &fdstr, &pidstr);
                if (r < 0)
                        return log_debug_errno(r, "Failed to deserialize pidref '%s': %m", e);
                if (r == 0)
                        return log_debug_errno(SYNTHETIC_ERRNO(EINVAL), "Cannot deserialize pidref from empty string.");

                assert(r <= 2);

                fd = deserialize_fd(fds, fdstr);
                if (fd < 0)
                        return fd;

                /* The serialization format changed after 255.4. In systemd <= 255.4 only pidfd is
                 * serialized, but that causes problems when reconstructing pidref (see serialize_pidref for
                 * details). After 255.4 the pid is serialized as well even if we have a pidfd, but we still
                 * need to support older format as we might be upgrading from a version that still uses the
                 * old format. */
                if (pidstr) {
                        pid_t pid;

                        r = parse_pid(pidstr, &pid);
                        if (r < 0)
                                return log_debug_errno(r, "Failed to parse PID: %s", pidstr);

                        *ret = (PidRef) {
                                .pid = pid,
                                .fd = TAKE_FD(fd),
                        };
                } else
                        r = pidref_set_pidfd_consume(ret, TAKE_FD(fd));
        } else {
                pid_t pid;

                r = parse_pid(value, &pid);
                if (r < 0)
                        return log_debug_errno(r, "Failed to parse PID: %s", value);

                r = pidref_set_pid(ret, pid);
        }
        if (r < 0)
                return log_debug_errno(r, "Failed to initialize pidref: %m");

        return 0;
}

void deserialize_ratelimit(RateLimit *rl, const char *name, const char *value) {
        usec_t begin, interval;
        unsigned num, burst;

        assert(rl);
        assert(name);
        assert(value);

        if (sscanf(value, USEC_FMT " " USEC_FMT " %u %u", &begin, &interval, &num, &burst) != 4)
                return log_notice("Failed to parse %s, ignoring: %s", name, value);

        /* Preserve the counter only if the configuration didn't change. */
        rl->num = (interval == rl->interval && burst == rl->burst) ? num : 0;
        rl->begin = begin;
}

int open_serialization_fd(const char *ident) {
        assert(ident);

        int fd = memfd_new_full(ident, MFD_ALLOW_SEALING);
        if (fd < 0)
                return fd;

        log_debug("Serializing %s to memfd.", ident);
        return fd;
}

int open_serialization_file(const char *ident, FILE **ret) {
        _cleanup_fclose_ FILE *f = NULL;
        _cleanup_close_ int fd;

        assert(ret);

        fd = open_serialization_fd(ident);
        if (fd < 0)
                return fd;

        f = take_fdopen(&fd, "w+");
        if (!f)
                return -errno;

        *ret = TAKE_PTR(f);
        return 0;
}

int finish_serialization_fd(int fd) {
        assert(fd >= 0);

        if (lseek(fd, 0, SEEK_SET) < 0)
                return -errno;

        return memfd_set_sealed(fd);
}

int finish_serialization_file(FILE *f) {
        int r;

        assert(f);

        r = fflush_and_check(f);
        if (r < 0)
                return r;

        if (fseeko(f, 0, SEEK_SET) < 0)
                return -errno;

        int fd = fileno(f);
        if (fd < 0)
                return -EBADF;

        return memfd_set_sealed(fd);
}

1	/* SPDX-License-Identifier: LGPL-2.1-or-later */
2
3	#include <fcntl.h>
4	#include <unistd.h>
5
6	#include "sd-id128.h"
7
8	#include "alloc-util.h"
9	#include "env-util.h"
10	#include "escape.h"
11	#include "extract-word.h"
12	#include "fd-util.h"
13	#include "fdset.h"
14	#include "fileio.h"
15	#include "format-util.h"
16	#include "hexdecoct.h"
17	#include "image-policy.h"
18	#include "log.h"
19	#include "memfd-util.h"
20	#include "parse-util.h"
21	#include "pidref.h"
22	#include "ratelimit.h"
23	#include "serialize.h"
24	#include "set.h"
25	#include "string-util.h"
26	#include "strv.h"
27	#include "time-util.h"
28
29	int serialize_item(FILE f, const char key, const char *value) {	548,751✔
30	assert(f);	548,751✔
31	assert(key);	548,751✔
32
33	if (!value)	548,751✔
34	return 0;
35
36	/* Make sure that anything we serialize we can also read back again with read_line() with a maximum line size
37	* of LONG_LINE_MAX. This is a safety net only. All code calling us should filter this out earlier anyway. */
38	if (strlen(key) + 1 + strlen(value) + 1 > LONG_LINE_MAX)	481,438✔
39	return log_warning_errno(SYNTHETIC_ERRNO(EINVAL), "Attempted to serialize overly long item '%s', refusing.", key);	7✔
40
41	fputs(key, f);	481,431✔
42	fputc('=', f);	481,431✔
43	fputs(value, f);	481,431✔
44	fputc('\n', f);	481,431✔
45
46	return 1;	481,431✔
47	}
48
49	int serialize_item_escaped(FILE f, const char key, const char *value) {	41,957✔
50	_cleanup_free_ char *c = NULL;	41,957✔
51
52	assert(f);	41,957✔
53	assert(key);	41,957✔
54
55	if (!value)	41,957✔
56	return 0;
57
58	c = xescape(value, " ");	28,595✔
59	if (!c)	28,595✔
60	return log_oom();	×
61
62	return serialize_item(f, key, c);	28,595✔
63	}
64
65	int serialize_item_format(FILE f, const char key, const char *format, ...) {	353,577✔
66	_cleanup_free_ char *allocated = NULL;	353,577✔
67	char buf[256]; /* Something reasonably short that fits nicely on any stack (i.e. is considerably less	353,577✔
68	* than LONG_LINE_MAX (1MiB!) */
69	const char *b;	353,577✔
70	va_list ap;	353,577✔
71	int k;	353,577✔
72
73	assert(f);	353,577✔
74	assert(key);	353,577✔
75	assert(format);	353,577✔
76
77	/* First, let's try to format this into a stack buffer */
78	va_start(ap, format);	353,577✔
79	k = vsnprintf(buf, sizeof(buf), format, ap);	353,577✔
80	va_end(ap);	353,577✔
81
82	if (k < 0)	353,577✔
83	return log_warning_errno(errno, "Failed to serialize item '%s', ignoring: %m", key);	×
84	if (strlen(key) + 1 + k + 1 > LONG_LINE_MAX) /* See above */	353,577✔
85	return log_warning_errno(SYNTHETIC_ERRNO(EINVAL), "Attempted to serialize overly long item '%s', refusing.", key);	×
86
87	if ((size_t) k < sizeof(buf))	353,577✔
88	b = buf; /* Yay, it fit! */
89	else {
90	/* So the string didn't fit in the short buffer above, but was not above our total limit,
91	* hence let's format it via dynamic memory */
92
93	va_start(ap, format);	×
94	k = vasprintf(&allocated, format, ap);	×
95	va_end(ap);	×
96
97	if (k < 0)	×
98	return log_warning_errno(errno, "Failed to serialize item '%s', ignoring: %m", key);	×
99
100	b = allocated;	×
101	}
102
103	fputs(key, f);	353,577✔
104	fputc('=', f);	353,577✔
105	fputs(b, f);	353,577✔
106	fputc('\n', f);	353,577✔
107
108	return 1;
109	}
110
111	int serialize_fd(FILE f, FDSet fds, const char *key, int fd) {	56,712✔
112	int copy;	56,712✔
113
114	assert(f);	56,712✔
115	assert(fds);	56,712✔
116	assert(key);	56,712✔
117
118	if (fd < 0)	56,712✔
119	return 0;
120
121	copy = fdset_put_dup(fds, fd);	7,954✔
122	if (copy < 0)	7,954✔
123	return log_error_errno(copy, "Failed to add file descriptor to serialization set: %m");	×
124
125	return serialize_item_format(f, key, "%i", copy);	7,954✔
126	}
127
128	int serialize_fd_many(FILE f, FDSet fds, const char *key, const int fd_array[], size_t n_fd_array) {	775✔
129	_cleanup_free_ char *t = NULL;	775✔
130
131	assert(f);	775✔
132
133	if (n_fd_array == 0)	775✔
134	return 0;
135
136	assert(fd_array);	775✔
137
138	for (size_t i = 0; i < n_fd_array; i++) {	2,067✔
139	int copy;	1,292✔
140
141	if (fd_array[i] < 0)	1,292✔
142	return -EBADF;
143
144	copy = fdset_put_dup(fds, fd_array[i]);	1,292✔
145	if (copy < 0)	1,292✔
146	return log_error_errno(copy, "Failed to add file descriptor to serialization set: %m");	×
147
148	if (strextendf_with_separator(&t, " ", "%i", copy) < 0)	1,292✔
149	return log_oom();	×
150	}
151
152	return serialize_item(f, key, t);	775✔
153	}
154
155	int serialize_usec(FILE f, const char key, usec_t usec) {	22,882✔
156	assert(f);	22,882✔
157	assert(key);	22,882✔
158
159	if (usec == USEC_INFINITY)	22,882✔
160	return 0;
161
162	return serialize_item_format(f, key, USEC_FMT, usec);	5,210✔
163	}
164
165	int serialize_dual_timestamp(FILE f, const char key, const dual_timestamp *t) {	175,069✔
166	assert(f);	175,069✔
167	assert(key);	175,069✔
168	assert(t);	175,069✔
169
170	if (!dual_timestamp_is_set(t))	175,069✔
171	return 0;
172
173	return serialize_item_format(f, key, USEC_FMT " " USEC_FMT, t->realtime, t->monotonic);	71,796✔
174	}
175
176	int serialize_strv(FILE f, const char key, char * const *l) {	44,175✔
177	int ret = 0, r;	44,175✔
178
179	/* Returns the first error, or positive if anything was serialized, 0 otherwise. */
180
181	assert(f);	44,175✔
182	assert(key);	44,175✔
183
184	STRV_FOREACH(i, l) {	71,343✔
185	r = serialize_item_escaped(f, key, *i);	27,168✔
186	if ((ret >= 0 && r < 0) \|\|	27,168✔
187	(ret == 0 && r > 0))	27,167✔
188	ret = r;	7,119✔
189	}
190
191	return ret;	44,175✔
192	}
193
194	int serialize_id128(FILE f, const char key, sd_id128_t id) {	22,801✔
195	assert(f);	22,801✔
196	assert(key);	22,801✔
197
198	if (sd_id128_is_null(id))	22,801✔
199	return 0;	8,348✔
200
201	return serialize_item_format(f, key, SD_ID128_FORMAT_STR, SD_ID128_FORMAT_VAL(id));	14,453✔
202	}
203
204	int serialize_pidref(FILE f, FDSet fds, const char key, PidRef pidref) {	14,487✔
205	int r;	14,487✔
206
207	assert(f);	14,487✔
208	assert(fds);	14,487✔
209
210	if (!pidref_is_set(pidref))	14,487✔
211	return 0;
212
213	/* We always serialize the pid separately, to keep downgrades mostly working (older versions will
214	* deserialize the pid and silently fail to deserialize the pidfd). If we also have a pidfd, we
215	* serialize both the pid and pidfd, so that we can construct the exact same pidref after
216	* deserialization (this doesn't work with only the pidfd, as we can't retrieve the original pid
217	* from the pidfd anymore if the process is reaped). */
218
219	if (pidref->fd >= 0) {	827✔
220	int copy = fdset_put_dup(fds, pidref->fd);	827✔
221	if (copy < 0)	827✔
222	return log_error_errno(copy, "Failed to add file descriptor to serialization set: %m");	×
223
224	r = serialize_item_format(f, key, "@%i:" PID_FMT, copy, pidref->pid);	827✔
225	if (r < 0)	827✔
226	return r;
227	}
228
229	return serialize_item_format(f, key, PID_FMT, pidref->pid);	827✔
230	}
231
232	int serialize_ratelimit(FILE f, const char key, const RateLimit *rl) {	48,810✔
233	assert(rl);	48,810✔
234
235	return serialize_item_format(f, key,	97,620✔
236	USEC_FMT " " USEC_FMT " %u %u",
237	rl->begin,	48,810✔
238	rl->interval,	48,810✔
239	rl->num,	48,810✔
240	rl->burst);	48,810✔
241	}
242
243	int serialize_item_hexmem(FILE f, const char key, const void *p, size_t l) {	2,483✔
244	_cleanup_free_ char *encoded = NULL;	2,483✔
245	int r;	2,483✔
246
247	assert(f);	2,483✔
248	assert(key);	2,483✔
249
250	if (!p && l > 0)	2,483✔
251	return -EINVAL;
252
253	if (l == 0)	2,483✔
254	return 0;
255
256	encoded = hexmem(p, l);	1✔
257	if (!encoded)	1✔
258	return log_oom_debug();	×
259
260	r = serialize_item(f, key, encoded);	1✔
261	if (r < 0)	1✔
262	return r;	×
263
264	return 1;
265	}
266
267	int serialize_item_base64mem(FILE f, const char key, const void *p, size_t l) {	2,483✔
268	_cleanup_free_ char *encoded = NULL;	2,483✔
269	ssize_t len;	2,483✔
270	int r;	2,483✔
271
272	assert(f);	2,483✔
273	assert(key);	2,483✔
274
275	if (!p && l > 0)	2,483✔
276	return -EINVAL;
277
278	if (l == 0)	2,483✔
279	return 0;
280
281	len = base64mem(p, l, &encoded);	1✔
282	if (len <= 0)	1✔
283	return log_oom_debug();	×
284
285	r = serialize_item(f, key, encoded);	1✔
286	if (r < 0)	1✔
287	return r;	×
288
289	return 1;
290	}
291
292	int serialize_string_set(FILE f, const char key, const Set *s) {	4,964✔
293	int r;	4,964✔
294
295	assert(f);	4,964✔
296	assert(key);	4,964✔
297
298	if (set_isempty(s))	4,964✔
299	return 0;	4,964✔
300
301	/* Serialize as individual items, as each element might contain separators and escapes */
302
303	const char *e;	1✔
304	SET_FOREACH(e, s) {	3✔
305	r = serialize_item(f, key, e);	2✔
306	if (r < 0)	2✔
307	return r;	×
308	}
309
310	return 1;	1✔
311	}
312
313	int serialize_image_policy(FILE f, const char key, const ImagePolicy *p) {	7,443✔
314	_cleanup_free_ char *policy = NULL;	7,443✔
315	int r;	7,443✔
316
317	assert(f);	7,443✔
318	assert(key);	7,443✔
319
320	if (!p)	7,443✔
321	return 0;
322
323	r = image_policy_to_string(p, /* simplify= */ false, &policy);	42✔
324	if (r < 0)	42✔
325	return r;
326
327	r = serialize_item(f, key, policy);	42✔
328	if (r < 0)	42✔
329	return r;	×
330
331	return 1;
332	}
333
334	int serialize_bool(FILE f, const char key, bool b) {	230,478✔
335	return serialize_item(f, key, yes_no(b));	430,275✔
336	}
337
338	int serialize_bool_elide(FILE f, const char key, bool b) {	114,181✔
339	return b ? serialize_item(f, key, yes_no(b)) : 0;	114,181✔
340	}
341
342	int deserialize_read_line(FILE f, char *ret) {	1,878,664✔
343	_cleanup_free_ char *line = NULL;	1,878,664✔
344	int r;	1,878,664✔
345
346	assert(f);	1,878,664✔
347	assert(ret);	1,878,664✔
348
349	r = read_stripped_line(f, LONG_LINE_MAX, &line);	1,878,664✔
350	if (r < 0)	1,878,664✔
351	return log_error_errno(r, "Failed to read serialization line: %m");	×
352	if (r == 0) { /* eof */	1,878,664✔
353	*ret = NULL;	31✔
354	return 0;	31✔
355	}
356
357	if (isempty(line)) { /* End marker */	1,878,633✔
358	*ret = NULL;	68,343✔
359	return 0;	68,343✔
360	}
361
362	*ret = TAKE_PTR(line);	1,810,290✔
363	return 1;	1,810,290✔
364	}
365
366	int deserialize_fd(FDSet fds, const char value) {	37,312✔
367	_cleanup_close_ int our_fd = -EBADF;	37,312✔
368	int parsed_fd;	37,312✔
369
370	assert(value);	37,312✔
371
372	parsed_fd = parse_fd(value);	37,312✔
373	if (parsed_fd < 0)	37,312✔
374	return log_debug_errno(parsed_fd, "Failed to parse file descriptor serialization: %s", value);	×
375
376	our_fd = fdset_remove(fds, parsed_fd); /* Take possession of the fd */	37,312✔
377	if (our_fd < 0)	37,312✔
378	return log_debug_errno(our_fd, "Failed to acquire fd from serialization fds: %m");	×
379
380	return TAKE_FD(our_fd);
381	}
382
383	int deserialize_fd_many(FDSet fds, const char value, size_t n, int *ret) {	2,075✔
384	int r, *fd_array = NULL;	2,075✔
385	size_t m = 0;	2,075✔
386
387	assert(value);	2,075✔
388
389	fd_array = new(int, n);	2,075✔
390	if (!fd_array)	2,075✔
391	return -ENOMEM;	2,075✔
392
393	CLEANUP_ARRAY(fd_array, m, close_many_and_free);	4,150✔
394
395	for (;;) {	11,067✔
396	_cleanup_free_ char *w = NULL;	4,496✔
397	int fd;	6,571✔
398
399	r = extract_first_word(&value, &w, NULL, 0);	6,571✔
400	if (r < 0)	6,571✔
401	return r;
402	if (r == 0) {	6,571✔
403	if (m < n) /* Too few */	2,075✔
404	return -EINVAL;
405
406	break;	2,075✔
407	}
408
409	if (m >= n) /* Too many */	4,496✔
410	return -EINVAL;
411
412	fd = deserialize_fd(fds, w);	4,496✔
413	if (fd < 0)	4,496✔
414	return fd;
415
416	fd_array[m++] = fd;	4,496✔
417	}
418
419	memcpy(ret, fd_array, m * sizeof(int));	2,075✔
420	fd_array = mfree(fd_array);	2,075✔
421
422	return 0;	2,075✔
423	}
424
425	int deserialize_strv(const char value, char **l) {	82,249✔
426	ssize_t unescaped_len;	82,249✔
427	char *unescaped;	82,249✔
428
429	assert(l);	82,249✔
430	assert(value);	82,249✔
431
432	unescaped_len = cunescape(value, 0, &unescaped);	82,249✔
433	if (unescaped_len < 0)	82,249✔
434	return unescaped_len;	×
435
436	return strv_consume(l, unescaped);	82,249✔
437	}
438
439	int deserialize_usec(const char value, usec_t ret) {	13,572✔
440	int r;	13,572✔
441
442	assert(value);	13,572✔
443	assert(ret);	13,572✔
444
445	r = safe_atou64(value, ret);	13,572✔
446	if (r < 0)	13,572✔
447	return log_debug_errno(r, "Failed to parse usec value \"%s\": %m", value);	×
448
449	return 0;
450	}
451
452	int deserialize_dual_timestamp(const char value, dual_timestamp ret) {	55,594✔
453	uint64_t a, b;	55,594✔
454	int r, pos;	55,594✔
455
456	assert(value);	55,594✔
457	assert(ret);	55,594✔
458
459	pos = strspn(value, WHITESPACE);	55,594✔
460	if (value[pos] == '-')	55,594✔
461	return -EINVAL;	55,594✔
462	pos += strspn(value + pos, DIGITS);	55,593✔
463	pos += strspn(value + pos, WHITESPACE);	55,593✔
464	if (value[pos] == '-')	55,593✔
465	return -EINVAL;
466
467	r = sscanf(value, "%" PRIu64 "%" PRIu64 "%n", &a, &b, &pos);	55,592✔
468	if (r != 2)	55,592✔
469	return log_debug_errno(SYNTHETIC_ERRNO(EINVAL),	1✔
470	"Failed to parse dual timestamp value \"%s\".",
471	value);
472
473	if (value[pos] != '\0')	55,591✔
474	/* trailing garbage */
475	return -EINVAL;
476
477	*ret = (dual_timestamp) {	55,590✔
478	.realtime = a,
479	.monotonic = b,
480	};
481
482	return 0;	55,590✔
483	}
484
485	int deserialize_environment(const char value, char **list) {	614✔
486	_cleanup_free_ char *unescaped = NULL;	614✔
487	ssize_t l;	614✔
488	int r;	614✔
489
490	assert(value);	614✔
491	assert(list);	614✔
492
493	/* Changes the environment strv inline. /
494
495	l = cunescape(value, 0, &unescaped);	614✔
496	if (l < 0)	614✔
497	return log_error_errno(l, "Failed to unescape: %m");	2✔
498
499	r = strv_env_replace_consume(list, TAKE_PTR(unescaped));	612✔
500	if (r < 0)	612✔
501	return log_error_errno(r, "Failed to append environment variable: %m");	×
502
503	return 0;
504	}
505
506	int deserialize_pidref(FDSet fds, const char value, PidRef *ret) {	671✔
507	const char *e;	671✔
508	int r;	671✔
509
510	assert(value);	671✔
511	assert(ret);	671✔
512
513	e = startswith(value, "@");	671✔
514	if (e) {	671✔
515	_cleanup_free_ char fdstr = NULL, pidstr = NULL;	671✔
516	_cleanup_close_ int fd = -EBADF;	671✔
517
518	r = extract_many_words(&e, ":", /* flags= */ 0, &fdstr, &pidstr);	671✔
519	if (r < 0)	671✔
520	return log_debug_errno(r, "Failed to deserialize pidref '%s': %m", e);	×
521	if (r == 0)	671✔
522	return log_debug_errno(SYNTHETIC_ERRNO(EINVAL), "Cannot deserialize pidref from empty string.");	×
523
524	assert(r <= 2);	671✔
525
526	fd = deserialize_fd(fds, fdstr);	671✔
527	if (fd < 0)	671✔
528	return fd;
529
530	/* The serialization format changed after 255.4. In systemd <= 255.4 only pidfd is
531	* serialized, but that causes problems when reconstructing pidref (see serialize_pidref for
532	* details). After 255.4 the pid is serialized as well even if we have a pidfd, but we still
533	* need to support older format as we might be upgrading from a version that still uses the
534	* old format. */
535	if (pidstr) {	671✔
536	pid_t pid;	671✔
537
538	r = parse_pid(pidstr, &pid);	671✔
539	if (r < 0)	671✔
540	return log_debug_errno(r, "Failed to parse PID: %s", pidstr);	×
541
542	*ret = (PidRef) {	671✔
543	.pid = pid,
544	.fd = TAKE_FD(fd),	671✔
545	};
546	} else
547	r = pidref_set_pidfd_consume(ret, TAKE_FD(fd));	×
548	} else {
UNCOV 549	pid_t pid;	×
550
UNCOV 551	r = parse_pid(value, &pid);	×
UNCOV 552	if (r < 0)	×
553	return log_debug_errno(r, "Failed to parse PID: %s", value);	×
554
UNCOV 555	r = pidref_set_pid(ret, pid);	×
556	}
557	if (r < 0)	671✔
558	return log_debug_errno(r, "Failed to initialize pidref: %m");	×
559
560	return 0;
561	}
562
563	void deserialize_ratelimit(RateLimit rl, const char name, const char *value) {	38,037✔
564	usec_t begin, interval;	38,037✔
565	unsigned num, burst;	38,037✔
566
567	assert(rl);	38,037✔
568	assert(name);	38,037✔
569	assert(value);	38,037✔
570
571	if (sscanf(value, USEC_FMT " " USEC_FMT " %u %u", &begin, &interval, &num, &burst) != 4)	38,037✔
572	return log_notice("Failed to parse %s, ignoring: %s", name, value);	×
573
574	/* Preserve the counter only if the configuration didn't change. */
575	rl->num = (interval == rl->interval && burst == rl->burst) ? num : 0;	38,037✔
576	rl->begin = begin;	38,037✔
577	}
578
579	int open_serialization_fd(const char *ident) {	4,800✔
580	assert(ident);	4,800✔
581
582	int fd = memfd_new_full(ident, MFD_ALLOW_SEALING);	4,800✔
583	if (fd < 0)	4,800✔
584	return fd;
585
586	log_debug("Serializing %s to memfd.", ident);	4,800✔
587	return fd;
588	}
589
590	int open_serialization_file(const char ident, FILE *ret) {	2,635✔
591	_cleanup_fclose_ FILE *f = NULL;	2,635✔
592	_cleanup_close_ int fd;	2,635✔
593
594	assert(ret);	2,635✔
595
596	fd = open_serialization_fd(ident);	2,635✔
597	if (fd < 0)	2,635✔
598	return fd;
599
600	f = take_fdopen(&fd, "w+");	2,635✔
601	if (!f)	2,635✔
602	return -errno;	×
603
604	*ret = TAKE_PTR(f);	2,635✔
605	return 0;	2,635✔
606	}
607
608	int finish_serialization_fd(int fd) {	2,165✔
609	assert(fd >= 0);	2,165✔
610
611	if (lseek(fd, 0, SEEK_SET) < 0)	2,165✔
612	return -errno;	×
613
614	return memfd_set_sealed(fd);	2,165✔
615	}
616
617	int finish_serialization_file(FILE *f) {	2,635✔
618	int r;	2,635✔
619
620	assert(f);	2,635✔
621
622	r = fflush_and_check(f);	2,635✔
623	if (r < 0)	2,635✔
624	return r;
625
626	if (fseeko(f, 0, SEEK_SET) < 0)	2,635✔
627	return -errno;	×
628
629	int fd = fileno(f);	2,635✔
630	if (fd < 0)	2,635✔
631	return -EBADF;
632
633	return memfd_set_sealed(fd);	2,635✔
634	}

systemd / systemd / 21770421410

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous