diff options
| author | Zhao Liu <zhao1.liu@intel.com> | 2024-03-12 14:03:37 +0800 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2024-03-12 11:45:45 +0100 |
| commit | 46ff64a826bbd70e98484c8c7f4c3c1d83363f1d (patch) | |
| tree | 05517ade1c9a4471589aa744ffd7989eda4821d8 /migration/options.c | |
| parent | 95e9053a34ca99b90f47073a30e02fa00afc3aeb (diff) | |
| download | qemu-46ff64a826bbd70e98484c8c7f4c3c1d83363f1d.zip qemu-46ff64a826bbd70e98484c8c7f4c3c1d83363f1d.tar.gz qemu-46ff64a826bbd70e98484c8c7f4c3c1d83363f1d.tar.bz2 | |
error: Move ERRP_GUARD() to the beginning of the function
Since the commit 05e385d2a9 ("error: Move ERRP_GUARD() to the beginning
of the function"), there are new codes that don't put ERRP_GUARD() at
the beginning of the functions.
As stated in the commit 05e385d2a9: "include/qapi/error.h advises to put
ERRP_GUARD() right at the beginning of the function, because only then
can it guard the whole function.", so clean up the few spots
disregarding the advice.
Inspired-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-ID: <20240312060337.3240965-1-zhao1.liu@linux.intel.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'migration/options.c')
| -rw-r--r-- | migration/options.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/migration/options.c b/migration/options.c index caff0a2..cf25326 100644 --- a/migration/options.c +++ b/migration/options.c @@ -478,9 +478,9 @@ static bool migrate_incoming_started(void) */ bool migrate_caps_check(bool *old_caps, bool *new_caps, Error **errp) { + ERRP_GUARD(); MigrationIncomingState *mis = migration_incoming_get_current(); - ERRP_GUARD(); #ifndef CONFIG_LIVE_BLOCK_MIGRATION if (new_caps[MIGRATION_CAPABILITY_BLOCK]) { error_setg(errp, "QEMU compiled without old-style (blk/-b, inc/-i) " |