]> xenbits.xensource.com Git - xen.git/commitdiff
libxl: use libxl_domain_config_init and not memset 0
authorIan Campbell <ian.campbell@citrix.com>
Wed, 25 Apr 2012 10:35:42 +0000 (11:35 +0100)
committerIan Campbell <ian.campbell@citrix.com>
Wed, 25 Apr 2012 10:35:42 +0000 (11:35 +0100)
I missed a couple of memsets in 25237:31489be80c51, we need to use
libxl_domain_config_init everywhere and not memset since not all fields are
initialised to zero now (the type field in particular). This fixes an abort
with "xl list <dom>" for a specific domain due to assert(type == -1) in
libxl_domain_build_info_init_type().

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
tools/libxl/xl_cmdimpl.c

index b81873ed5abbe37b399b1e7f7364e5fea0bb4fa1..ff82a35cd6cb7b65a17e4fcf06bc3d34e1ca0703 100644 (file)
@@ -2464,7 +2464,7 @@ static void list_domains_details(const libxl_dominfo *info, int nb_domain)
         if (rc)
             continue;
         CHK_ERRNO(asprintf(&config_file, "<domid %d data>", info[i].domid));
-        memset(&d_config, 0x00, sizeof(d_config));
+        libxl_domain_config_init(&d_config);
         parse_config_data(config_file, (char *)data, len, &d_config);
         printf_info(default_output_format, info[i].domid, &d_config);
         libxl_domain_config_dispose(&d_config);
@@ -3546,7 +3546,7 @@ int main_config_update(int argc, char **argv)
         exit(1);
     }
 
-    memset(&d_config, 0x00, sizeof(d_config));
+    libxl_domain_config_init(&d_config);
 
     parse_config_data(filename, config_data, config_len, &d_config);