On 3/24/20 11:36 AM, Vladimir Sementsov-Ogievskiy wrote: > local_err is used again in mirror_exit_common() after > bdrv_set_backing_hd(), so we must zero it. Otherwise try to set > non-NULL local_err will crash. > > Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> > --- > block/mirror.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/block/mirror.c b/block/mirror.c > index 447051dbc6..6203e5946e 100644 > --- a/block/mirror.c > +++ b/block/mirror.c > @@ -678,6 +678,7 @@ static int mirror_exit_common(Job *job) > bdrv_set_backing_hd(target_bs, backing, &local_err); > if (local_err) { > error_report_err(local_err); > + local_err = NULL; > ret = -EPERM; > } > } > Reviewed-by: John Snow <js...@redhat.com>
- [PATCH for-5.0 0/6] Several error use-after-f... Vladimir Sementsov-Ogievskiy
- [PATCH 2/6] block/mirror: fix use after ... Vladimir Sementsov-Ogievskiy
- Re: [PATCH 2/6] block/mirror: fix us... Eric Blake
- Re: [PATCH 2/6] block/mirror: fix us... John Snow
- Re: [PATCH 2/6] block/mirror: fix us... Max Reitz
- Re: [PATCH 2/6] block/mirror: fi... Max Reitz
- Re: [PATCH 2/6] block/mirror: fi... Vladimir Sementsov-Ogievskiy
- Re: [PATCH 2/6] block/mirror... Max Reitz
- Re: [PATCH 2/6] block/m... Markus Armbruster
- Re: [PATCH 2/6] block/mirror: fi... Eric Blake
- Re: [PATCH 2/6] block/mirror: fix us... Max Reitz
- [PATCH 5/6] migration/ram: fix use after... Vladimir Sementsov-Ogievskiy
- Re: [PATCH 5/6] migration/ram: fix u... Dr. David Alan Gilbert
- [PATCH 1/6] scripts/coccinelle: add erro... Vladimir Sementsov-Ogievskiy