Expecting every pointer member from s->parameters to be freed individually is prone to leave some fields forgotten when the code is eventually updated. Use a dealloc visitor to free them all at once.
Signed-off-by: Fabiano Rosas <[email protected]> --- migration/options.c | 24 +++++++++++++++++------- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/migration/options.c b/migration/options.c index 733aae51a8..cc5a66750c 100644 --- a/migration/options.c +++ b/migration/options.c @@ -1002,11 +1002,21 @@ AnnounceParameters *migrate_announce_params(void) return ≈ } -static void migrate_tls_opts_free(MigrationParameters *params) +static bool migrate_params_free(MigrationParameters *params, Error **errp) { - qapi_free_StrOrNull(params->tls_creds); - qapi_free_StrOrNull(params->tls_hostname); - qapi_free_StrOrNull(params->tls_authz); + Visitor *v = qapi_dealloc_visitor_new(); + bool ret; + + /* + * qapi_free_MigrationParameters can't be used here because + * MigrationParameters is embedded in MigrationState due to qdev + * needing to access the offset of the migration properties inside + * the migration object. + */ + ret = visit_type_MigrationParameters_members(v, params, errp); + visit_free(v); + + return ret; } /* normalize QTYPE_QNULL to QTYPE_QSTRING "" */ @@ -1290,9 +1300,9 @@ void qmp_migrate_set_parameters(MigrationParameters *params, Error **errp) return; } - migrate_tls_opts_free(cur); - qapi_free_BitmapMigrationNodeAliasList(cur->block_bitmap_mapping); - qapi_free_strList(cur->cpr_exec_command); + if (!migrate_params_free(cur, errp)) { + return; + } /* mark all present, so they're all copied */ migrate_mark_all_params_present(tmp); -- 2.51.0
