[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/6] dump/win_dump: fix use after free of err
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 3/6] dump/win_dump: fix use after free of err |
Date: |
Mon, 30 Mar 2020 17:54:54 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Vladimir Sementsov-Ogievskiy <address@hidden> writes:
> It's possible that we'll try to set err twice (or more). It's bad, it
> will crash.
True.
> Instead, use warn_report().
Improvement even without the potential crash enabled by the loop.
> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> ---
> dump/win_dump.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/dump/win_dump.c b/dump/win_dump.c
> index eda2a48974..652c7bad99 100644
> --- a/dump/win_dump.c
> +++ b/dump/win_dump.c
> @@ -304,13 +304,11 @@ static void restore_context(WinDumpHeader64 *h,
> struct saved_context *saved_ctx)
> {
> int i;
> - Error *err = NULL;
>
> for (i = 0; i < h->NumberProcessors; i++) {
> if (cpu_memory_rw_debug(first_cpu, saved_ctx[i].addr,
> (uint8_t *)&saved_ctx[i].ctx, sizeof(WinContext), 1)) {
> - error_setg(&err, "win-dump: failed to restore CPU #%d context",
> i);
> - warn_report_err(err);
> + warn_report("win-dump: failed to restore CPU #%d context", i);
> }
> }
> }
Reviewed-by: Markus Armbruster <address@hidden>
Re: [PATCH 2/6] block/mirror: fix use after free of local_err, Max Reitz, 2020/03/25
[PATCH 3/6] dump/win_dump: fix use after free of err, Vladimir Sementsov-Ogievskiy, 2020/03/24
- Re: [PATCH 3/6] dump/win_dump: fix use after free of err,
Markus Armbruster <=
[PATCH 4/6] migration/colo: fix use after free of local_err, Vladimir Sementsov-Ogievskiy, 2020/03/24
[PATCH 5/6] migration/ram: fix use after free of local_err, Vladimir Sementsov-Ogievskiy, 2020/03/24
[PATCH 6/6] qga/commands-posix: fix use after free of local_err, Vladimir Sementsov-Ogievskiy, 2020/03/24
Re: [PATCH 6/6] qga/commands-posix: fix use after free of local_err, Markus Armbruster, 2020/03/31