[PATCH 00/20] qemu: migration_cookie: Refactor and modernize

[Michal Prívozník]

On 10/2/20 10:57 AM, Peter Krempa wrote:
> Note that this applies on top of the recent bitmap refactors
> 
> Peter Krempa (20):
>    qemu: migration_cookie: Extract parsing/validation of mandatory
>      features
>    qemuMigrationCookieXMLParse: Switch to single-purpose temporary
>      variables
>    qemuMigrationCookieXMLParse: Check domain element count more
>      defensively
>    qemuMigrationCookieXMLParse: Decrease scope of 'nodes' and use
>      automatic freeing
>    qemuMigrationCookieXMLParse: Remove comment mentioning that error was
>      already set
>    qemuMigrationCookieXMLParse: Remove 'error' label
>    qemuMigrationCookieGraphicsXMLFormat: Use 'virXMLFormatElement'
>    qemuMigrationCookieNetworkXMLFormat: Refactor XML formatting
>    qemuMigrationCookieXMLFormat: Extract formatting of NBD
>    qemuDomainExtractTLSSubject: Refactor memory handling
>    qemu: migration_cookie: Register 'autoptr' functions for internal
>      types
>    qemuMigrationCookieGraphicsSpiceAlloc: Refactor memory handling
>    qemuMigrationCookieNetworkAlloc: Refactor memory handling
>    qemuMigrationCookieXMLFormat: Refactor memory handling
>    qemuMigrationCookieNetworkXMLParse: Refactor memory handling
>    qemuMigrationCookieNBDXMLParse: Refactor memory handling
>    qemuMigrationCookieCapsXMLParse: Refactor memory handling
>    qemuMigrationCookieAddCaps: Use 'g_new0' instead of VIR_ALLOC
>    qemuMigrationCookieXMLParse: Avoid VIR_FREE when parsing lockstate
>    qemu: migration_cookie: s/VIR_FREE/g_free/
> 
>   src/qemu/qemu_migration_cookie.c | 493 ++++++++++++++-----------------
>   1 file changed, 214 insertions(+), 279 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

Michal