summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYu Watanabe <watanabe.yu+github@gmail.com>2021-06-15 07:09:29 +0200
committerYu Watanabe <watanabe.yu+github@gmail.com>2021-06-15 07:09:29 +0200
commit7802194ac0286c75e39f74a1624764eb2aa0fd7a (patch)
treea735c8ceb30419e39d9e36420cecde5f45e7187b
parentMerge pull request #19913 from yuwata/network-fix-counter (diff)
downloadsystemd-7802194ac0286c75e39f74a1624764eb2aa0fd7a.tar.xz
systemd-7802194ac0286c75e39f74a1624764eb2aa0fd7a.zip
tree-wide: add missing whitespace at the end of comments
-rw-r--r--src/basic/process-util.c2
-rw-r--r--src/basic/random-util.c2
-rw-r--r--src/basic/socket-util.c4
-rw-r--r--src/basic/time-util.c10
-rw-r--r--src/core/execute.c6
-rw-r--r--src/core/manager.c6
-rw-r--r--src/core/mount.c2
-rw-r--r--src/core/unit.c2
-rw-r--r--src/coredump/stacktrace.c2
-rw-r--r--src/cryptsetup/cryptsetup.c2
-rw-r--r--src/home/homed-home.h2
-rw-r--r--src/home/homework-luks.c2
-rw-r--r--src/home/user-record-util.c2
-rw-r--r--src/journal/journald-server.c2
-rw-r--r--src/libsystemd/sd-bus/sd-bus.c2
-rw-r--r--src/libsystemd/sd-journal/journal-file.c2
-rw-r--r--src/login/user-runtime-dir.c2
-rw-r--r--src/network/networkd-link.c2
-rw-r--r--src/network/networkd-lldp-rx.c2
-rw-r--r--src/network/networkd-manager.h2
-rw-r--r--src/network/networkd-network.h2
-rw-r--r--src/network/networkd-setlink.c2
-rw-r--r--src/nspawn/nspawn.c2
-rw-r--r--src/partition/repart.c2
-rw-r--r--src/resolve/resolved-dns-query.c6
-rw-r--r--src/shared/format-table.c2
-rw-r--r--src/shutdown/umount.c2
-rw-r--r--src/sysusers/sysusers.c2
-rw-r--r--src/test/test-fs-util.c2
-rw-r--r--src/test/test-path-util.c2
-rw-r--r--src/test/test-psi-util.c2
-rw-r--r--src/udev/udev-builtin-input_id.c2
-rw-r--r--src/udev/udev-builtin-path_id.c2
-rw-r--r--src/udev/udevd.c2
34 files changed, 45 insertions, 45 deletions
diff --git a/src/basic/process-util.c b/src/basic/process-util.c
index 22903cd677..20b2441468 100644
--- a/src/basic/process-util.c
+++ b/src/basic/process-util.c
@@ -820,7 +820,7 @@ int wait_for_terminate_with_timeout(pid_t pid, usec_t timeout) {
/* Assuming we woke due to the child exiting. */
if (waitid(P_PID, pid, &status, WEXITED|WNOHANG) == 0) {
if (status.si_pid == pid) {
- /* This is the correct child.*/
+ /* This is the correct child. */
if (status.si_code == CLD_EXITED)
return (status.si_status == 0) ? 0 : -EPROTO;
else
diff --git a/src/basic/random-util.c b/src/basic/random-util.c
index a125e5228e..c2be962355 100644
--- a/src/basic/random-util.c
+++ b/src/basic/random-util.c
@@ -188,7 +188,7 @@ int genuine_random_bytes(void *p, size_t n, RandomFlags flags) {
* invocations or so. That's because we don't really care about the quality here. We
* generally prefer using RDRAND if the caller allows us to, since this way we won't upset
* the kernel's random subsystem by accessing it before the pool is initialized (after all it
- * will kmsg log about every attempt to do so)..*/
+ * will kmsg log about every attempt to do so). */
for (;;) {
unsigned long u;
size_t m;
diff --git a/src/basic/socket-util.c b/src/basic/socket-util.c
index 552ec053ff..f6e751a09f 100644
--- a/src/basic/socket-util.c
+++ b/src/basic/socket-util.c
@@ -107,7 +107,7 @@ int socket_address_verify(const SocketAddress *a, bool strict) {
/* If there's no embedded NUL byte, then the size needs to match the whole
* structure or the structure with one extra NUL byte suffixed. (Yeah, Linux is awful,
* and considers both equivalent: getsockname() even extends sockaddr_un beyond its
- * size if the path is non NUL terminated.)*/
+ * size if the path is non NUL terminated.) */
if (!IN_SET(a->size, sizeof(a->sockaddr.un.sun_path), sizeof(a->sockaddr.un.sun_path)+1))
return -EINVAL;
}
@@ -1192,7 +1192,7 @@ int sockaddr_un_set_path(struct sockaddr_un *ret, const char *path) {
/* Don't allow paths larger than the space in sockaddr_un. Note that we are a tiny bit more restrictive than
* the kernel is: we insist on NUL termination (both for abstract namespace and regular file system socket
* addresses!), which the kernel doesn't. We do this to reduce chance of incompatibility with other apps that
- * do not expect non-NUL terminated file system path*/
+ * do not expect non-NUL terminated file system path. */
if (l+1 > sizeof(ret->sun_path))
return -EINVAL;
diff --git a/src/basic/time-util.c b/src/basic/time-util.c
index da3f025583..26a6762b17 100644
--- a/src/basic/time-util.c
+++ b/src/basic/time-util.c
@@ -1030,7 +1030,7 @@ int parse_time(const char *t, usec_t *usec, usec_t default_unit) {
s = extract_multiplier(p + strspn(p, WHITESPACE), &multiplier);
if (s == p && *s != '\0')
- /* Don't allow '12.34.56', but accept '12.34 .56' or '12.34s.56'*/
+ /* Don't allow '12.34.56', but accept '12.34 .56' or '12.34s.56' */
return -EINVAL;
p = s;
@@ -1058,7 +1058,7 @@ int parse_time(const char *t, usec_t *usec, usec_t default_unit) {
r += k;
}
- /* Don't allow "0.-0", "3.+1", "3. 1", "3.sec" or "3.hoge"*/
+ /* Don't allow "0.-0", "3.+1", "3. 1", "3.sec" or "3.hoge" */
if (b == e + 1)
return -EINVAL;
}
@@ -1205,7 +1205,7 @@ int parse_nsec(const char *t, nsec_t *nsec) {
s = extract_nsec_multiplier(p + strspn(p, WHITESPACE), &multiplier);
if (s == p && *s != '\0')
- /* Don't allow '12.34.56', but accept '12.34 .56' or '12.34s.56'*/
+ /* Don't allow '12.34.56', but accept '12.34 .56' or '12.34s.56' */
return -EINVAL;
p = s;
@@ -1233,7 +1233,7 @@ int parse_nsec(const char *t, nsec_t *nsec) {
r += k;
}
- /* Don't allow "0.-0", "3.+1", "3. 1", "3.sec" or "3.hoge"*/
+ /* Don't allow "0.-0", "3.+1", "3. 1", "3.sec" or "3.hoge" */
if (b == e + 1)
return -EINVAL;
}
@@ -1438,7 +1438,7 @@ int get_timezone(char **ret) {
r = readlink_malloc("/etc/localtime", &t);
if (r == -ENOENT) {
- /* If the symlink does not exist, assume "UTC", like glibc does*/
+ /* If the symlink does not exist, assume "UTC", like glibc does */
z = strdup("UTC");
if (!z)
return -ENOMEM;
diff --git a/src/core/execute.c b/src/core/execute.c
index 5bb41db888..f9d23272c4 100644
--- a/src/core/execute.c
+++ b/src/core/execute.c
@@ -2450,7 +2450,7 @@ static int setup_exec_directory(
/* Then, change the ownership of the whole tree, if necessary. When dynamic users are used we
* drop the suid/sgid bits, since we really don't want SUID/SGID files for dynamic UID/GID
- * assignments to exist.*/
+ * assignments to exist. */
r = path_chown_recursive(pp ?: p, uid, gid, context->dynamic_user ? 01777 : 07777);
if (r < 0)
goto fail;
@@ -2886,7 +2886,7 @@ static int setup_credentials(
* Yes it's nasty playing games with /dev/ and /dev/shm/ like this, since it does not exist
* for this purpose, but there are few other candidates that work equally well for us, and
* given that the we do this in a privately namespaced short-lived single-threaded process
- * that no one else sees this should be OK to do.*/
+ * that no one else sees this should be OK to do. */
r = mount_nofollow_verbose(LOG_DEBUG, NULL, "/dev", NULL, MS_SLAVE|MS_REC, NULL); /* Turn off propagation from our namespace to host */
if (r < 0)
@@ -3836,7 +3836,7 @@ static int exec_child(
_cleanup_strv_free_ char **suggested_paths = NULL;
/* On top of that, make sure we bypass our own NSS module nss-systemd comprehensively for any NSS
- * checks, if DynamicUser=1 is used, as we shouldn't create a feedback loop with ourselves here.*/
+ * checks, if DynamicUser=1 is used, as we shouldn't create a feedback loop with ourselves here. */
if (putenv((char*) "SYSTEMD_NSS_DYNAMIC_BYPASS=1") != 0) {
*exit_status = EXIT_USER;
return log_unit_error_errno(unit, errno, "Failed to update environment: %m");
diff --git a/src/core/manager.c b/src/core/manager.c
index addd04067f..21358a0469 100644
--- a/src/core/manager.c
+++ b/src/core/manager.c
@@ -3791,7 +3791,7 @@ int manager_reload(Manager *m) {
/* Start by flushing out all jobs and units, all generated units, all runtime environments, all dynamic users
* and everything else that is worth flushing out. We'll get it all back from the serialization — if we need
- * it.*/
+ * it. */
manager_clear_jobs_and_units(m);
lookup_paths_flush_generator(&m->lookup_paths);
@@ -3927,7 +3927,7 @@ static void manager_notify_finished(Manager *m) {
if (dual_timestamp_is_set(&m->timestamps[MANAGER_TIMESTAMP_INITRD])) {
- /* The initrd case on bare-metal*/
+ /* The initrd case on bare-metal */
kernel_usec = m->timestamps[MANAGER_TIMESTAMP_INITRD].monotonic - m->timestamps[MANAGER_TIMESTAMP_KERNEL].monotonic;
initrd_usec = m->timestamps[MANAGER_TIMESTAMP_USERSPACE].monotonic - m->timestamps[MANAGER_TIMESTAMP_INITRD].monotonic;
@@ -3943,7 +3943,7 @@ static void manager_notify_finished(Manager *m) {
format_timespan(userspace, sizeof(userspace), userspace_usec, USEC_PER_MSEC),
format_timespan(sum, sizeof(sum), total_usec, USEC_PER_MSEC)));
} else {
- /* The initrd-less case on bare-metal*/
+ /* The initrd-less case on bare-metal */
kernel_usec = m->timestamps[MANAGER_TIMESTAMP_USERSPACE].monotonic - m->timestamps[MANAGER_TIMESTAMP_KERNEL].monotonic;
initrd_usec = 0;
diff --git a/src/core/mount.c b/src/core/mount.c
index 8e39cf3ab3..ee4dd0eea1 100644
--- a/src/core/mount.c
+++ b/src/core/mount.c
@@ -1584,7 +1584,7 @@ static int mount_setup_new_unit(
/* This unit was generated because /proc/self/mountinfo reported it. Remember this, so that by the time we load
* the unit file for it (and thus add in extra deps right after) we know what source to attributes the deps
- * to.*/
+ * to. */
MOUNT(u)->from_proc_self_mountinfo = true;
/* We have only allocated the stub now, let's enqueue this unit for loading now, so that everything else is
diff --git a/src/core/unit.c b/src/core/unit.c
index 5f4739b32a..4dc6ff08d6 100644
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -2075,7 +2075,7 @@ bool unit_is_bound_by_inactive(Unit *u, Unit **ret_culprit) {
assert(u);
/* Checks whether this unit is bound to another unit that is inactive, i.e. whether we should stop
- * because the other unit is down.*/
+ * because the other unit is down. */
if (unit_active_state(u) != UNIT_ACTIVE || u->job) {
/* Don't clean up while the unit is transitioning or is even inactive. */
diff --git a/src/coredump/stacktrace.c b/src/coredump/stacktrace.c
index 102ad2e65b..0edb1b40a7 100644
--- a/src/coredump/stacktrace.c
+++ b/src/coredump/stacktrace.c
@@ -287,7 +287,7 @@ static int module_callback(Dwfl_Module *mod, void **userdata, const char *name,
GElf_Phdr mem, *program_header;
Elf_Data *data;
- /* The core file stores the ELF files in the PT_LOAD segment .*/
+ /* The core file stores the ELF files in the PT_LOAD segment. */
program_header = gelf_getphdr(c->elf, i, &mem);
if (!program_header || program_header->p_type != PT_LOAD)
continue;
diff --git a/src/cryptsetup/cryptsetup.c b/src/cryptsetup/cryptsetup.c
index 64a7f5bc37..0fa41b8360 100644
--- a/src/cryptsetup/cryptsetup.c
+++ b/src/cryptsetup/cryptsetup.c
@@ -1109,7 +1109,7 @@ static int attach_luks_or_plain_or_bitlk_by_tpm2(
&keyslot,
&token);
if (r == -ENXIO)
- /* No further TPM2 tokens found in the LUKS2 header.*/
+ /* No further TPM2 tokens found in the LUKS2 header. */
return log_debug_errno(SYNTHETIC_ERRNO(EAGAIN),
found_some
? "No TPM2 metadata matching the current system state found in LUKS2 header, falling back to traditional unlocking."
diff --git a/src/home/homed-home.h b/src/home/homed-home.h
index 2afff0c023..2fba86f700 100644
--- a/src/home/homed-home.h
+++ b/src/home/homed-home.h
@@ -84,7 +84,7 @@ struct Home {
/* Note that the 'state' field is only set to a state while we are doing something (i.e. activating,
* deactivating, creating, removing, and such), or when the home is an "unfixated" one. When we are
* done with an operation we invalidate the state. This is hint for home_get_state() to check the
- * state on request as needed from the mount table and similar.*/
+ * state on request as needed from the mount table and similar. */
HomeState state;
int signed_locally; /* signed only by us */
diff --git a/src/home/homework-luks.c b/src/home/homework-luks.c
index 87c999b6e3..6448883fe0 100644
--- a/src/home/homework-luks.c
+++ b/src/home/homework-luks.c
@@ -40,7 +40,7 @@
/* Round down to the nearest 1K size. Note that Linux generally handles block devices with 512 blocks only,
* but actually doesn't accept uneven numbers in many cases. To avoid any confusion around this we'll
- * strictly round disk sizes down to the next 1K boundary.*/
+ * strictly round disk sizes down to the next 1K boundary. */
#define DISK_SIZE_ROUND_DOWN(x) ((x) & ~UINT64_C(1023))
int run_mark_dirty(int fd, bool b) {
diff --git a/src/home/user-record-util.c b/src/home/user-record-util.c
index b205f1d1e6..b5ed6b1689 100644
--- a/src/home/user-record-util.c
+++ b/src/home/user-record-util.c
@@ -199,7 +199,7 @@ int user_record_reconcile(
if (!user_record_compatible(host, embedded))
return -EREMCHG;
- /* Embedded identities may not contain secrets or binding info*/
+ /* Embedded identities may not contain secrets or binding info */
if ((embedded->mask & (USER_RECORD_SECRET|USER_RECORD_BINDING)) != 0)
return -EINVAL;
diff --git a/src/journal/journald-server.c b/src/journal/journald-server.c
index 2907e26b77..30f04f5383 100644
--- a/src/journal/journald-server.c
+++ b/src/journal/journald-server.c
@@ -242,7 +242,7 @@ void server_space_usage_message(Server *s, JournalStorage *storage) {
static bool uid_for_system_journal(uid_t uid) {
- /* Returns true if the specified UID shall get its data stored in the system journal*/
+ /* Returns true if the specified UID shall get its data stored in the system journal. */
return uid_is_system(uid) || uid_is_dynamic(uid) || uid == UID_NOBODY;
}
diff --git a/src/libsystemd/sd-bus/sd-bus.c b/src/libsystemd/sd-bus/sd-bus.c
index 090288af0d..a32e2f5e20 100644
--- a/src/libsystemd/sd-bus/sd-bus.c
+++ b/src/libsystemd/sd-bus/sd-bus.c
@@ -1526,7 +1526,7 @@ int bus_set_address_machine(sd_bus *b, bool user, const char *machine) {
* since the --user socket path depends on $XDG_RUNTIME_DIR which is set via PAM. Thus, to be
* able to connect, we need to have a PAM session. Our way out? We use systemd-run to get
* into the container and acquire a PAM session there, and then invoke systemd-stdio-bridge
- * in it, which propagates the bus transport to us.*/
+ * in it, which propagates the bus transport to us. */
if (rhs) {
if (rhs > machine)
diff --git a/src/libsystemd/sd-journal/journal-file.c b/src/libsystemd/sd-journal/journal-file.c
index a67aa0f803..ff2d23f480 100644
--- a/src/libsystemd/sd-journal/journal-file.c
+++ b/src/libsystemd/sd-journal/journal-file.c
@@ -2854,7 +2854,7 @@ int journal_file_compare_locations(JournalFile *af, JournalFile *bf) {
assert(bf->location_type == LOCATION_SEEK);
/* If contents, timestamps and seqnum match, these entries are
- * identical*/
+ * identical. */
if (sd_id128_equal(af->current_boot_id, bf->current_boot_id) &&
af->current_monotonic == bf->current_monotonic &&
af->current_realtime == bf->current_realtime &&
diff --git a/src/login/user-runtime-dir.c b/src/login/user-runtime-dir.c
index 9b5bdcc3ff..07d72d214d 100644
--- a/src/login/user-runtime-dir.c
+++ b/src/login/user-runtime-dir.c
@@ -167,7 +167,7 @@ static int do_umount(const char *user) {
int r;
/* The user may be already removed. So, first try to parse the string by parse_uid(),
- * and if it fails, fall back to get_user_creds().*/
+ * and if it fails, fall back to get_user_creds(). */
if (parse_uid(user, &uid) < 0) {
r = get_user_creds(&user, &uid, NULL, NULL, NULL, 0);
if (r < 0)
diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c
index a3a2fb8aac..c98f684bfd 100644
--- a/src/network/networkd-link.c
+++ b/src/network/networkd-link.c
@@ -2126,7 +2126,7 @@ static int link_update_name(Link *link, sd_netlink_message *message) {
r = sd_netlink_message_read_string(message, IFLA_IFNAME, &ifname);
if (r == -ENODATA)
- /* Hmm?? But ok.*/
+ /* Hmm?? But ok. */
return 0;
if (r < 0)
return log_link_debug_errno(link, r, "Failed to read interface name in RTM_NEWLINK message: %m");
diff --git a/src/network/networkd-lldp-rx.c b/src/network/networkd-lldp-rx.c
index 56e4244f41..c30cc36b51 100644
--- a/src/network/networkd-lldp-rx.c
+++ b/src/network/networkd-lldp-rx.c
@@ -41,7 +41,7 @@ static bool link_lldp_rx_enabled(Link *link) {
/* LLDP should be handled on bridge and bond slaves as those have a direct connection to their peers,
* not on the bridge/bond master. Linux doesn't even (by default) forward lldp packets to the bridge
- * master.*/
+ * master. */
if (link->kind && STR_IN_SET(link->kind, "bridge", "bond"))
return false;
diff --git a/src/network/networkd-manager.h b/src/network/networkd-manager.h
index c7fea16ab2..24845a55c2 100644
--- a/src/network/networkd-manager.h
+++ b/src/network/networkd-manager.h
@@ -85,7 +85,7 @@ struct Manager {
Hashmap *route_table_numbers_by_name;
Hashmap *route_table_names_by_number;
- /* For link speed meter*/
+ /* For link speed meter */
bool use_speed_meter;
sd_event_source *speed_meter_event_source;
usec_t speed_meter_interval_usec;
diff --git a/src/network/networkd-network.h b/src/network/networkd-network.h
index 341cf477ce..98b8b95295 100644
--- a/src/network/networkd-network.h
+++ b/src/network/networkd-network.h
@@ -160,7 +160,7 @@ struct Network {
OrderedHashmap *dhcp_client_send_options;
OrderedHashmap *dhcp_client_send_vendor_options;
- /* DHCPv6 Client support*/
+ /* DHCPv6 Client support */
bool dhcp6_use_address;
bool dhcp6_use_dns;
bool dhcp6_use_dns_set;
diff --git a/src/network/networkd-setlink.c b/src/network/networkd-setlink.c
index dea353dfbb..126a14b8bc 100644
--- a/src/network/networkd-setlink.c
+++ b/src/network/networkd-setlink.c
@@ -390,7 +390,7 @@ static int link_configure(
return log_link_debug_errno(link, r, "Could not open IFLA_AF_SPEC container: %m");
if (!link->network->bridge) {
- /* master needs BRIDGE_FLAGS_SELF flag*/
+ /* master needs BRIDGE_FLAGS_SELF flag */
r = sd_netlink_message_append_u16(req, IFLA_BRIDGE_FLAGS, BRIDGE_FLAGS_SELF);
if (r < 0)
return log_link_debug_errno(link, r, "Could not append IFLA_BRIDGE_FLAGS attribute: %m");
diff --git a/src/nspawn/nspawn.c b/src/nspawn/nspawn.c
index 21aa4f246f..c0a1da3851 100644
--- a/src/nspawn/nspawn.c
+++ b/src/nspawn/nspawn.c
@@ -3688,7 +3688,7 @@ static int outer_child(
* --volatile=) without this interfering with our ability to access files such as
* /etc/localtime to copy into the container. Note that we use a fixed place for this
* (instead of a temporary directory, since we are living in our own mount namspace here
- * already, and thus don't need to be afraid of colliding with anyone else's mounts).*/
+ * already, and thus don't need to be afraid of colliding with anyone else's mounts). */
(void) mkdir_p("/run/systemd/nspawn-root", 0755);
r = mount_nofollow_verbose(LOG_ERR, "/", "/run/systemd/nspawn-root", NULL, MS_BIND|MS_REC, NULL);
diff --git a/src/partition/repart.c b/src/partition/repart.c
index 824cfe8cf7..f4ead0a31b 100644
--- a/src/partition/repart.c
+++ b/src/partition/repart.c
@@ -4952,7 +4952,7 @@ static int run(int argc, char *argv[]) {
r = context_open_copy_block_paths(
context,
arg_root,
- loop_device ? loop_device->devno : /* if --image= is specified, only allow partitions on the loopback device*/
+ loop_device ? loop_device->devno : /* if --image= is specified, only allow partitions on the loopback device */
arg_root && !arg_image ? 0 : /* if --root= is specified, don't accept any block device */
(dev_t) -1); /* if neither is specified, make no restrictions */
if (r < 0)
diff --git a/src/resolve/resolved-dns-query.c b/src/resolve/resolved-dns-query.c
index 2a048f1070..7f341986d9 100644
--- a/src/resolve/resolved-dns-query.c
+++ b/src/resolve/resolved-dns-query.c
@@ -1102,7 +1102,7 @@ int dns_query_process_cname_one(DnsQuery *q) {
* Hence we first check of the answers we collected are sufficient to answer all our questions
* directly. If one question wasn't answered we go on, waiting for more replies. However, if there's
* a CNAME/DNAME response we use it, and redirect to it, regardless if it was a response to the A or
- * the AAAA query.*/
+ * the AAAA query. */
DNS_QUESTION_FOREACH(k, question) {
bool match = false;
@@ -1276,12 +1276,12 @@ bool dns_query_fully_authoritative(DnsQuery *q) {
/* We are authoritative for everything synthetic (except if a previous CNAME/DNAME) wasn't
* synthetic. (Note: SD_RESOLVED_SYNTHETIC is reset on each CNAME/DNAME, hence the explicit check for
- * previous synthetic DNAME/CNAME redirections.)*/
+ * previous synthetic DNAME/CNAME redirections.) */
if ((q->answer_query_flags & SD_RESOLVED_SYNTHETIC) && !q->previous_redirect_non_synthetic)
return true;
/* We are also authoritative for everything coming only from the trust anchor and the local
* zones. (Note: the SD_RESOLVED_FROM_xyz flags we merge on each redirect, hence no need to
- * explicitly check previous redirects here.)*/
+ * explicitly check previous redirects here.) */
return (q->answer_query_flags & SD_RESOLVED_FROM_MASK & ~(SD_RESOLVED_FROM_TRUST_ANCHOR | SD_RESOLVED_FROM_ZONE)) == 0;
}
diff --git a/src/shared/format-table.c b/src/shared/format-table.c
index 8b1240d63a..85d38ae8f8 100644
--- a/src/shared/format-table.c
+++ b/src/shared/format-table.c
@@ -2136,7 +2136,7 @@ int table_print(Table *t, FILE *f) {
/* Never give more than requested. If we hit a column like this, there's more
* space to allocate to other columns which means we need to restart the
* iteration. However, if we hit a column like this, let's assign it the space
- * it wanted for good early.*/
+ * it wanted for good early. */
w = requested_width[j];
restart = true;
diff --git a/src/shutdown/umount.c b/src/shutdown/umount.c
index 906756292b..c2a26242c0 100644
--- a/src/shutdown/umount.c
+++ b/src/shutdown/umount.c
@@ -560,7 +560,7 @@ static int umount_with_timeout(MountPoint *m, int umount_log_level) {
* rather than blocking indefinitely. If the filesysten is
* "busy", this may allow processes to die, thus making the
* filesystem less busy so the unmount might succeed (rather
- * then return EBUSY).*/
+ * than return EBUSY). */
r = umount2(m->path, MNT_FORCE);
if (r < 0)
log_full_errno(umount_log_level, errno, "Failed to unmount %s: %m", m->path);
diff --git a/src/sysusers/sysusers.c b/src/sysusers/sysusers.c
index d76556e75d..4131a26c10 100644
--- a/src/sysusers/sysusers.c
+++ b/src/sysusers/sysusers.c
@@ -281,7 +281,7 @@ static int make_backup(const char *target, const char *x) {
if (rename(dst_tmp, backup) < 0)
return errno;
- dst_tmp = mfree(dst_tmp); /* disable the unlink_and_freep() hook now that the file has been renamed*/
+ dst_tmp = mfree(dst_tmp); /* disable the unlink_and_freep() hook now that the file has been renamed */
return 0;
}
diff --git a/src/test/test-fs-util.c b/src/test/test-fs-util.c
index 63bb830370..a0f91c35d1 100644
--- a/src/test/test-fs-util.c
+++ b/src/test/test-fs-util.c
@@ -827,7 +827,7 @@ static void test_path_is_encrypted(void) {
int booted = sd_booted(); /* If this is run in build environments such as koji, /dev might be a
* reguar fs. Don't assume too much if not running under systemd. */
- log_info("/* %s (sd_booted=%d)*/", __func__, booted);
+ log_info("/* %s (sd_booted=%d) */", __func__, booted);
test_path_is_encrypted_one("/home", -1);
test_path_is_encrypted_one("/var", -1);
diff --git a/src/test/test-path-util.c b/src/test/test-path-util.c
index 622e38c1bb..50aacdb967 100644
--- a/src/test/test-path-util.c
+++ b/src/test/test-path-util.c
@@ -913,7 +913,7 @@ static void test_filename_is_valid(void) {
}
static void test_path_is_valid_and_safe_one(const char *p, bool ret) {
- log_debug("/* %s(\"%s\")*/", __func__, strnull(p));
+ log_debug("/* %s(\"%s\") */", __func__, strnull(p));
assert_se(path_is_valid(p) == ret);
if (ret)
diff --git a/src/test/test-psi-util.c b/src/test/test-psi-util.c
index 0b5a30cf57..c636cf255b 100644
--- a/src/test/test-psi-util.c
+++ b/src/test/test-psi-util.c
@@ -23,7 +23,7 @@ static void test_read_mem_pressure(void) {
assert_se(write_string_file(path, "herpdederp\n", WRITE_STRING_FILE_CREATE) == 0);
assert_se(read_resource_pressure(path, PRESSURE_TYPE_SOME, &rp) < 0);
- /* Pressure file with some invalid values*/
+ /* Pressure file with some invalid values */
assert_se(write_string_file(path, "some avg10=0.22=55 avg60=0.17=8 avg300=1.11=00 total=58761459\n"
"full avg10=0.23=55 avg60=0.16=8 avg300=1.08=00 total=58464525", WRITE_STRING_FILE_CREATE) == 0);
assert_se(read_resource_pressure(path, PRESSURE_TYPE_SOME, &rp) < 0);
diff --git a/src/udev/udev-builtin-input_id.c b/src/udev/udev-builtin-input_id.c
index 6a4911ca34..2331525c87 100644
--- a/src/udev/udev-builtin-input_id.c
+++ b/src/udev/udev-builtin-input_id.c
@@ -35,7 +35,7 @@ struct range {
unsigned end;
};
-/* key code ranges above BTN_MISC (start is inclusive, stop is exclusive)*/
+/* key code ranges above BTN_MISC (start is inclusive, stop is exclusive) */
static const struct range high_key_blocks[] = {
{ KEY_OK, BTN_DPAD_UP },
{ KEY_ALS_TOGGLE, BTN_TRIGGER_HAPPY }
diff --git a/src/udev/udev-builtin-path_id.c b/src/udev/udev-builtin-path_id.c
index 09cc1c3bd8..5e7945081e 100644
--- a/src/udev/udev-builtin-path_id.c
+++ b/src/udev/udev-builtin-path_id.c
@@ -282,7 +282,7 @@ static sd_device *handle_scsi_ata(sd_device *parent, char **path, char **compat_
return NULL;
if (bus != 0)
- /* Devices behind port multiplier have a bus != 0*/
+ /* Devices behind port multiplier have a bus != 0 */
path_prepend(path, "ata-%s.%u.0", port_no, bus);
else
/* Master/slave are distinguished by target id */
diff --git a/src/udev/udevd.c b/src/udev/udevd.c
index 5574abce9d..81d488a704 100644
--- a/src/udev/udevd.c
+++ b/src/udev/udevd.c
@@ -560,7 +560,7 @@ static int worker_main(Manager *_manager, sd_device_monitor *monitor, sd_device
if (r < 0)
log_debug_errno(r, "Failed to reset OOM score, ignoring: %m");
- /* Clear unnecessary data in Manager object.*/
+ /* Clear unnecessary data in Manager object. */
manager_clear_for_worker(manager);
r = sd_event_new(&manager->event);