src/boot/efi/measure.c

@@ -302,8 +302,9 @@ EFI_STATUS tpm_log_event(UINT32 pcrindex, const EFI_PHYSICAL_ADDRESS buffer, UIN
         EFI_TCG2 *tpm2;
 
         tpm2 = tcg2_interface_check();
-        if (tpm2)
+        if (tpm2) {
                 return tpm2_measure_to_pcr_and_event_log(tpm2, pcrindex, buffer, buffer_size, description);
+        }
 
         tpm1 = tcg1_interface_check();
         if (tpm1)

src/core/transaction.c

@@ -336,8 +336,9 @@ static char* merge_unit_ids(const char* unit_log_field, char **pairs) {
 
         STRV_FOREACH_PAIR(unit_id, job_type, pairs) {
                 next = strlen(unit_log_field) + strlen(*unit_id);
-                if (!GREEDY_REALLOC(ans, alloc, size + next + 1))
+                if (!GREEDY_REALLOC(ans, alloc, size + next + 1)) {
                         return mfree(ans);
+                }
 
                 sprintf(ans + size, "%s%s", unit_log_field, *unit_id);
                 if (*(unit_id+1))

src/home/homectl.c

@@ -535,8 +535,9 @@ static int inspect_home(int argc, char *argv[], void *userdata) {
                         }
 
                         r = bus_call_method(bus, &home_mgr, "GetUserRecordByName", &error, &reply, "s", *i);
-                } else
+                } else {
                         r = bus_call_method(bus, &home_mgr, "GetUserRecordByUID", &error, &reply, "u", (uint32_t) uid);
+                }
 
                 if (r < 0) {
                         log_error_errno(r, "Failed to inspect home: %s", bus_error_message(&error, r));

src/import/curl-util.c

@@ -166,8 +166,9 @@ CurlGlue *curl_glue_unref(CurlGlue *g) {
         if (g->curl)
                 curl_multi_cleanup(g->curl);
 
-        while ((io = hashmap_steal_first(g->ios)))
+        while ((io = hashmap_steal_first(g->ios))) {
                 sd_event_source_unref(io);
+        }
 
         hashmap_free(g->ios);
 

src/journal/test-catalog.c

@@ -116,8 +116,9 @@ static void test_catalog_import_merge(void) {
         h = test_import(input, -1, 0);
         assert_se(ordered_hashmap_size(h) == 1);
 
-        ORDERED_HASHMAP_FOREACH(payload, h, j)
+        ORDERED_HASHMAP_FOREACH(payload, h, j) {
                 assert_se(streq(combined, payload));
+        }
 }
 
 static void test_catalog_import_merge_no_body(void) {
@@ -148,8 +149,9 @@ static void test_catalog_import_merge_no_body(void) {
         h = test_import(input, -1, 0);
         assert_se(ordered_hashmap_size(h) == 1);
 
-        ORDERED_HASHMAP_FOREACH(payload, h, j)
+        ORDERED_HASHMAP_FOREACH(payload, h, j) {
                 assert_se(streq(combined, payload));
+        }
 }
 
 static void test_catalog_update(const char *database) {

src/libsystemd-network/dhcp6-option.c

@@ -464,8 +464,8 @@ int dhcp6_option_parse_ia(DHCP6Option *iaoption, DHCP6IA *ia) {
                         if (status < 0)
                                 return status;
                         if (status > 0) {
-                                log_dhcp6_client(client, "IA status %s",
+                                log_dhcp6_client(client, "IA status %d",
-                                                 dhcp6_message_status_to_string(status));
+                                                 status);
 
                                 return -EINVAL;
                         }

src/libsystemd-network/dhcp6-protocol.h

@@ -89,8 +89,7 @@ enum {
         DHCP6_STATUS_NO_BINDING                 = 3,
         DHCP6_STATUS_NOT_ON_LINK                = 4,
         DHCP6_STATUS_USE_MULTICAST              = 5,
-        DHCP6_STATUS_NO_PREFIX_AVAIL            = 6,
+        _DHCP6_STATUS_MAX                       = 6,
-        _DHCP6_STATUS_MAX                       = 7,
 };
 
 enum {

src/libsystemd-network/sd-dhcp6-client.c

@@ -112,7 +112,6 @@ const char * dhcp6_message_status_table[_DHCP6_STATUS_MAX] = {
         [DHCP6_STATUS_NO_BINDING] = "Binding unavailable",
         [DHCP6_STATUS_NOT_ON_LINK] = "Not on link",
         [DHCP6_STATUS_USE_MULTICAST] = "Use multicast",
-        [DHCP6_STATUS_NO_PREFIX_AVAIL] = "No prefix available",
 };
 
 DEFINE_STRING_TABLE_LOOKUP(dhcp6_message_status, int);

src/locale/keymap-util.c

@@ -267,8 +267,9 @@ int x11_read_data(Context *c, sd_bus_message *m) {
                                 else if (streq(a[1], "XkbOptions"))
                                         p = &c->x11_options;
 
-                                if (p)
+                                if (p) {
                                         free_and_replace(*p, a[2]);
+                                }
                         }
 
                 } else if (!in_section && first_word(l, "Section")) {

src/network/netdev/vxlan.c

@@ -38,14 +38,7 @@ static int netdev_vxlan_fill_message_create(NetDev *netdev, Link *link, sd_netli
                         return log_netdev_error_errno(netdev, r, "Could not append IFLA_VXLAN_ID attribute: %m");
         }
 
-        if (in_addr_is_null(v->group_family, &v->group) == 0) {
+        if (in_addr_is_null(v->remote_family, &v->remote) == 0) {
-                if (v->group_family == AF_INET)
-                        r = sd_netlink_message_append_in_addr(m, IFLA_VXLAN_GROUP, &v->group.in);
-                else
-                        r = sd_netlink_message_append_in6_addr(m, IFLA_VXLAN_GROUP6, &v->group.in6);
-                if (r < 0)
-                        return log_netdev_error_errno(netdev, r, "Could not append IFLA_VXLAN_GROUP attribute: %m");
-        } else  if (in_addr_is_null(v->remote_family, &v->remote) == 0) {
                 if (v->remote_family == AF_INET)
                         r = sd_netlink_message_append_in_addr(m, IFLA_VXLAN_GROUP, &v->remote.in);
                 else
@@ -355,11 +348,6 @@ static int netdev_vxlan_verify(NetDev *netdev, const char *filename) {
         if (!v->dest_port && v->generic_protocol_extension)
                 v->dest_port = 4790;
 
-        if (in_addr_is_null(v->group_family, &v->group) == 0 && in_addr_is_null(v->remote_family, &v->remote) == 0)
-                return log_netdev_warning_errno(netdev, SYNTHETIC_ERRNO(EINVAL),
-                                                "%s: VXLAN both 'Group=' and 'Remote=' cannot be specified. Ignoring.",
-                                                filename);
-
         return 0;
 }
 

src/network/networkctl.c

@@ -1549,17 +1549,9 @@ static int link_status_one(
                 }
 
                 if (IN_SET(info->vxlan_info.group_family, AF_INET, AF_INET6)) {
-                        const char *p;
-
-                        r = in_addr_is_multicast(info->vxlan_info.group_family, &info->vxlan_info.group);
-                        if (r <= 0)
-                                p = "Remote:";
-                        else
-                                p = "Group:";
-
                         r = table_add_many(table,
                                            TABLE_EMPTY,
-                                           TABLE_STRING, p,
+                                           TABLE_STRING, "Group:",
                                            info->vxlan_info.group_family == AF_INET ? TABLE_IN_ADDR : TABLE_IN6_ADDR,
                                            &info->vxlan_info.group);
                         if (r < 0)

src/test/test-strv.c

@@ -408,8 +408,9 @@ static void test_strv_split_newlines(void) {
         l = strv_split_newlines(str);
         assert_se(l);
 
-        STRV_FOREACH(s, l)
+        STRV_FOREACH(s, l) {
                 assert_se(streq(*s, input_table_multiple[i++]));
+        }
 }
 
 static void test_strv_split_nulstr(void) {

src/udev/ata_id/ata_id.c

@@ -514,8 +514,9 @@ int main(int argc, char *argv[]) {
                                 printf("ID_TYPE=generic\n");
                                 break;
                         }
-                } else
+                } else {
                         printf("ID_TYPE=disk\n");
+                }
                 printf("ID_BUS=ata\n");
                 printf("ID_MODEL=%s\n", model);
                 printf("ID_MODEL_ENC=%s\n", model_enc);
@@ -523,8 +524,9 @@ int main(int argc, char *argv[]) {
                 if (serial[0] != '\0') {
                         printf("ID_SERIAL=%s_%s\n", model, serial);
                         printf("ID_SERIAL_SHORT=%s\n", serial);
-                } else
+                } else {
                         printf("ID_SERIAL=%s\n", model);
+                }
 
                 if (id.command_set_1 & (1<<5)) {
                         printf("ID_ATA_WRITE_CACHE=1\n");

src/udev/scsi_id/scsi_id.c

@@ -111,8 +111,9 @@ static char *get_value(char **buffer) {
                  */
                 (*buffer)++;
                 end = quote_string;
-        } else
+        } else {
                 end = comma_string;
+        }
         val = strsep(buffer, end);
         if (val && end == quote_string)
                 /*

src/udev/udev-builtin-input_id.c

@@ -228,8 +228,9 @@ static bool test_pointers(sd_device *dev,
                         is_touchscreen = true;
                 else if (has_joystick_axes_or_buttons)
                         is_joystick = true;
-        } else if (has_joystick_axes_or_buttons)
+        } else if (has_joystick_axes_or_buttons) {
                 is_joystick = true;
+        }
 
         if (has_mt_coordinates) {
                 if (stylus_or_pen)

src/update-utmp/update-utmp.c

@@ -125,8 +125,9 @@ static int on_reboot(Context *c) {
 #if HAVE_AUDIT
         if (c->audit_fd >= 0)
                 if (audit_log_user_comm_message(c->audit_fd, AUDIT_SYSTEM_BOOT, "", "systemd-update-utmp", NULL, NULL, NULL, 1) < 0 &&
-                    errno != EPERM)
+                    errno != EPERM) {
                         r = log_error_errno(errno, "Failed to send audit message: %m");
+                }
 #endif
 
         /* If this call fails it will return 0, which
@@ -153,8 +154,9 @@ static int on_shutdown(Context *c) {
 #if HAVE_AUDIT
         if (c->audit_fd >= 0)
                 if (audit_log_user_comm_message(c->audit_fd, AUDIT_SYSTEM_SHUTDOWN, "", "systemd-update-utmp", NULL, NULL, NULL, 1) < 0 &&
-                    errno != EPERM)
+                    errno != EPERM) {
                         r = log_error_errno(errno, "Failed to send audit message: %m");
+                }
 #endif
 
         q = utmp_put_shutdown();