diff options
author | Eduardo Habkost <ehabkost@redhat.com> | 2017-06-08 10:39:05 -0300 |
---|---|---|
committer | Juan Quintela <quintela@redhat.com> | 2017-06-13 11:00:44 +0200 |
commit | 250561e1aebf69e911992da9017322df7aeaa564 (patch) | |
tree | e15752ffd84005f6717619fb8ba3a7cf8a5484c9 /migration/migration.c | |
parent | 0425dc976212a38961f0469003074d1df65a61f2 (diff) |
migration: Don't try to set *errp directly
Assigning directly to *errp is not valid, as errp may be NULL,
&error_fatal, or &error_abort. Use error_propagate() instead.
Cc: Juan Quintela <quintela@redhat.com>
Cc: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Diffstat (limited to 'migration/migration.c')
-rw-r--r-- | migration/migration.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/migration/migration.c b/migration/migration.c index 9dfa19abe1..ae087df56b 100644 --- a/migration/migration.c +++ b/migration/migration.c @@ -1179,7 +1179,7 @@ bool migration_is_blocked(Error **errp) } if (migration_blockers) { - *errp = error_copy(migration_blockers->data); + error_propagate(errp, error_copy(migration_blockers->data)); return true; } |