diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2023-02-03 16:21:41 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2023-02-23 19:48:57 +0100 |
| commit | 32125b14606a454ed109ea6d9da32c747e94926f (patch) | |
| tree | 7b79b418c3e67fa454905fc95a8b74cbe0c2e407 /scripts/qapi/expr.py | |
| parent | 10e5d70787b0bcb8c4449ee2fa6a0154d4138186 (diff) | |
| download | focaccia-qemu-32125b14606a454ed109ea6d9da32c747e94926f.tar.gz focaccia-qemu-32125b14606a454ed109ea6d9da32c747e94926f.zip | |
mirror: Fix access of uninitialised fields during start
bdrv_mirror_top_pwritev() accesses the job object when active mirroring is enabled. It disables this code during early initialisation while s->job isn't set yet. However, s->job is still set way too early when the job object isn't fully initialised. For example, &s->ops_in_flight isn't initialised yet and the in_flight bitmap doesn't exist yet. This causes crashes when a write request comes in too early. Move the assignment of s->job to when the mirror job is actually fully initialised to make sure that the mirror_top driver doesn't access it too early. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Message-Id: <20230203152202.49054-3-kwolf@redhat.com> Reviewed-by: Emanuele Giuseppe Esposito <eesposit@redhat.com> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'scripts/qapi/expr.py')
0 files changed, 0 insertions, 0 deletions