[v2] migration: A few fixes on coverity reports

[PATCH v2 0/4] migration: A few fixes on coverity reports

Peter Xu posted 4 patches 3 weeks, 2 days ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20251021220407.2662288-1-peterx@redhat.com

Maintainers: Steve Sistare <steven.sistare@oracle.com>, Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>

include/migration/cpr.h |  4 ++--
migration/cpr-exec.c    | 14 ++++++++++----
migration/cpr.c         | 15 +++++++++------
migration/migration.c   |  8 +++++++-
migration/savevm.c      |  7 ++++---
5 files changed, 32 insertions(+), 16 deletions(-)

Expand all Fold all

[PATCH v2 0/4] migration: A few fixes on coverity reports

Posted by Peter Xu 3 weeks, 2 days ago

v2:
- Added tags for Fabiano
- Patch 1: dump the %d too along with %s [Fabiano]
- Patch 2: make the error returned to the caller, with some refactoring
- Added new patch 4 to fix a crash on cpr-exec when args==NULL

A small series that fixes three coverity reported issues.  Please review,
thanks.

Peter Xu (4):
  migration: Fix error leak in postcopy_ram_listen_thread()
  migration/cpr: Fix coverity report in cpr_exec_persist_state()
  migration/cpr: Fix UAF in cpr_exec_cb() when execvp() fails
  migration/cpr: Avoid crashing QEMU when cpr-exec runs with no args

 include/migration/cpr.h |  4 ++--
 migration/cpr-exec.c    | 14 ++++++++++----
 migration/cpr.c         | 15 +++++++++------
 migration/migration.c   |  8 +++++++-
 migration/savevm.c      |  7 ++++---
 5 files changed, 32 insertions(+), 16 deletions(-)

-- 
2.50.1