[libvirt] [PATCH 1/2] qemu: Copy complete domain def in qemuDomainDefFormatBuf
Michal Privoznik
mprivozn at redhat.com
Mon Jul 11 06:18:57 UTC 2016
On 08.07.2016 17:40, Jiri Denemark wrote:
> Playing directly with our live definition, updating it, and reverting it
> back once we are done is very nice and it's quite dangerous too. Let's
> just make a copy of the domain definition if needed and do all tricks on
> the copy.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=1320470
>
> Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
> ---
> src/conf/domain_conf.c | 10 ++++++----
> src/qemu/qemu_domain.c | 44 ++++++++++++++++++++++++--------------------
> 2 files changed, 30 insertions(+), 24 deletions(-)
It's our own stupidity that we've ever allowed update flags for format
APIs. Now we have to be extra cautious about it and use hacks like this.
ACK though.
Michal
More information about the libvir-list
mailing list