Fix use-after-free in block job reconnection code

[PATCH 0/3] Fix use-after-free in block job reconnection code

Peter Krempa posted 3 patches 4 years ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1585226343.git.pkrempa@redhat.com

src/qemu/qemu_blockjob.c  |  9 +++------
src/qemu/qemu_blockjob.h  |  7 ++++---
src/qemu/qemu_migration.c | 17 +++++++----------
3 files changed, 14 insertions(+), 19 deletions(-)

Expand all Fold all

[PATCH 0/3] Fix use-after-free in block job reconnection code

Posted by Peter Krempa 4 years ago

Peter Krempa (3):
  qemu: migration: Don't use return value of qemuBlockJobUpdate
  qemuBlockJobUpdate: Remove return value
  qemuBlockJobRefreshJobs: Warn readers that 'job' may be invalid after
    update

 src/qemu/qemu_blockjob.c  |  9 +++------
 src/qemu/qemu_blockjob.h  |  7 ++++---
 src/qemu/qemu_migration.c | 17 +++++++----------
 3 files changed, 14 insertions(+), 19 deletions(-)

-- 
2.24.1

Re: [PATCH 0/3] Fix use-after-free in block job reconnection code

Posted by Michal Prívozník 4 years ago

On 26. 3. 2020 13:39, Peter Krempa wrote:
> Peter Krempa (3):
>   qemu: migration: Don't use return value of qemuBlockJobUpdate
>   qemuBlockJobUpdate: Remove return value
>   qemuBlockJobRefreshJobs: Warn readers that 'job' may be invalid after
>     update
> 
>  src/qemu/qemu_blockjob.c  |  9 +++------
>  src/qemu/qemu_blockjob.h  |  7 ++++---
>  src/qemu/qemu_migration.c | 17 +++++++----------
>  3 files changed, 14 insertions(+), 19 deletions(-)
> 
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal