[Xen-changelog] [xen master] xl: Fix assertion on domain reboot with new configuration

[patchbot@xxxxxxx]

commit dd75ed5c14704862b547412086cde5dcc671997e
Author:     Fatih Acar <fatih.acar@xxxxxxxxx>
AuthorDate: Mon Jan 30 15:33:18 2017 +0100
Commit:     Wei Liu <wei.liu2@xxxxxxxxxx>
CommitDate: Thu Feb 2 12:33:08 2017 +0000

    xl: Fix assertion on domain reboot with new configuration
    
    libxl_domain_build_info_dispose is not resetting the type field to 
LIBXL_DOMAIN_TYPE_INVALID.
    Instead, it is memseting the struct to 0 thus when 
libxl_domain_build_info_init_type is called
    after a dispose on the same struct, an assertion is triggered because type 
!= LIBXL_DOMAIN_TYPE_INVALID.
    Calling libxl_domain_build_info_init makes sure the type field is correctly 
initialized.
    
    Signed-off-by: Fatih Acar <fatih.acar@xxxxxxxxx>
    Signed-off-by: Nikita Kozlov <nikita.kozlov@xxxxxxxxx>
    Signed-off-by: Vincent Legout <vincent.legout@xxxxxxxxx>
    Signed-off-by: Baptiste Daroussin <baptiste.daroussin@xxxxxxxxx>
    Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>
---
 tools/libxl/xl_cmdimpl.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
index 7e8a8ae..196b8a6 100644
--- a/tools/libxl/xl_cmdimpl.c
+++ b/tools/libxl/xl_cmdimpl.c
@@ -2535,6 +2535,7 @@ static void reload_domain_config(uint32_t domid,
     if (t_len > 0) {
         LOG("\"xl\" configuration found, using it\n");
         libxl_domain_config_dispose(d_config);
+        libxl_domain_config_init(d_config);
         parse_config_data("<updated>", (const char *)t_data,
                           t_len, d_config);
         free(t_data);
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog