diff options
| author | Alberto Garcia <berto@igalia.com> | 2019-05-13 16:46:18 +0300 |
|---|---|---|
| committer | Max Reitz <mreitz@redhat.com> | 2019-05-28 20:30:55 +0200 |
| commit | b441dc71c0b7e8f488a6ebc2aa781b08a3a05038 (patch) | |
| tree | dd5875c418d32a8fb09dab833cd5e5921b56e325 /hw/core/qdev-properties-system.c | |
| parent | dd4118c792a8c2a104fe90274e8a41e0db1ebc56 (diff) | |
| download | focaccia-qemu-b441dc71c0b7e8f488a6ebc2aa781b08a3a05038.tar.gz focaccia-qemu-b441dc71c0b7e8f488a6ebc2aa781b08a3a05038.zip | |
block: Make bdrv_root_attach_child() unref child_bs on failure
A consequence of the previous patch is that bdrv_attach_child()
transfers the reference to child_bs from the caller to parent_bs,
which will drop it on bdrv_close() or when someone calls
bdrv_unref_child().
But this only happens when bdrv_attach_child() succeeds. If it fails
then the caller is responsible for dropping the reference to child_bs.
This patch makes bdrv_attach_child() take the reference also when
there is an error, freeing the caller for having to do it.
A similar situation happens with bdrv_root_attach_child(), so the
changes on this patch affect both functions.
Signed-off-by: Alberto Garcia <berto@igalia.com>
Message-id: 20dfb3d9ccec559cdd1a9690146abad5d204a186.1557754872.git.berto@igalia.com
[mreitz: Removed now superfluous BdrvChild * variable in
bdrv_open_child()]
Signed-off-by: Max Reitz <mreitz@redhat.com>
Diffstat (limited to 'hw/core/qdev-properties-system.c')
0 files changed, 0 insertions, 0 deletions