]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
machine: trivial coding style cleanups
authorYu Watanabe <watanabe.yu+github@gmail.com>
Tue, 15 Oct 2024 04:17:29 +0000 (13:17 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Tue, 15 Oct 2024 05:24:58 +0000 (14:24 +0900)
src/machine/machined-varlink.c

index 4b167c8fac30d6edb859283b78764d2525288641..536aee507d8c7204fadc97ea025aba0b05989217 100644 (file)
@@ -561,7 +561,8 @@ static int vl_method_list_images(sd_varlink *link, sd_json_variant *parameters,
         struct params {
                 const char *image_name;
                 bool acquire_metadata;
-        };
+        } p = {};
+        int r;
 
         static const sd_json_dispatch_field dispatch_table[] = {
                 { "name",            SD_JSON_VARIANT_STRING,    sd_json_dispatch_const_string, offsetof(struct params, image_name),        0 },
@@ -570,11 +571,6 @@ static int vl_method_list_images(sd_varlink *link, sd_json_variant *parameters,
                 {}
         };
 
-        _cleanup_hashmap_free_ Hashmap *images = NULL;
-        struct params p = {};
-        Image *image;
-        int r;
-
         assert(link);
         assert(parameters);
 
@@ -600,7 +596,7 @@ static int vl_method_list_images(sd_varlink *link, sd_json_variant *parameters,
         if (!FLAGS_SET(flags, SD_VARLINK_METHOD_MORE))
                 return sd_varlink_error(link, SD_VARLINK_ERROR_EXPECTED_MORE, NULL);
 
-        images = hashmap_new(&image_hash_ops);
+        _cleanup_hashmap_free_ Hashmap *images = hashmap_new(&image_hash_ops);
         if (!images)
                 return -ENOMEM;
 
@@ -608,7 +604,7 @@ static int vl_method_list_images(sd_varlink *link, sd_json_variant *parameters,
         if (r < 0)
                 return log_debug_errno(r, "Failed to discover images: %m");
 
-        Image *previous = NULL;
+        Image *image, *previous = NULL;
         HASHMAP_FOREACH(image, images) {
                 if (previous) {
                         r = list_image_one_and_maybe_read_metadata(link, previous, /* more = */ true, p.acquire_metadata);