[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 6/6] qga/commands-posix: fix use after free of local_err
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 6/6] qga/commands-posix: fix use after free of local_err |
Date: |
Tue, 31 Mar 2020 13:46:30 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Vladimir Sementsov-Ogievskiy <address@hidden> writes:
[...]
> I agree that this is a strange function and its logic is weird. But I
> don't know what the logic should be. My patch is still valid to just
> fix obvious use-after-free and possible leak. It doesn't fix the
> logic.
I sketched improved logic elsewhere in this thread, and I can turn that
into a patch.
I can either make it replace Vladimir's patch, or make it go on top. If
the latter, we can apply just Vladimir's patch for 5.0, and punt mine to
5.1
Got a preference?
- Re: [PATCH 1/6] scripts/coccinelle: add error-use-after-free.cocci, (continued)
- [PATCH 4/6] migration/colo: fix use after free of local_err, Vladimir Sementsov-Ogievskiy, 2020/03/24
- [PATCH 3/6] dump/win_dump: fix use after free of 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
Re: [PATCH for-5.0 0/6] Several error use-after-free, Richard Henderson, 2020/03/24