diff options
author | Stefano Garzarella <sgarzare@redhat.com> | 2023-08-03 10:28:24 +0200 |
---|---|---|
committer | Stefan Hajnoczi <stefanha@redhat.com> | 2023-08-03 11:28:43 -0400 |
commit | 0b054b4c82ebad6e90111e3ffa3514f841dbb1d0 (patch) | |
tree | 24f7a54328ff63f3a916aca3a17f8ee360ef22b1 /block | |
parent | 9ba37026fcf6b7f3f096c0cca3e1e7307802486b (diff) | |
download | qemu-0b054b4c82ebad6e90111e3ffa3514f841dbb1d0.zip qemu-0b054b4c82ebad6e90111e3ffa3514f841dbb1d0.tar.gz qemu-0b054b4c82ebad6e90111e3ffa3514f841dbb1d0.tar.bz2 |
block/blkio: close the fd when blkio_connect() fails
libblkio drivers take ownership of `fd` only after a successful
blkio_connect(), so if it fails, we are still the owners.
Fixes: cad2ccc395 ("block/blkio: use qemu_open() to support fd passing for virtio-blk")
Suggested-by: Hanna Czenczek <hreitz@redhat.com>
Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
Reviewed-by: Hanna Czenczek <hreitz@redhat.com>
Message-id: 20230803082825.25293-2-sgarzare@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'block')
-rw-r--r-- | block/blkio.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/block/blkio.c b/block/blkio.c index 8e7ce42..baba2f0 100644 --- a/block/blkio.c +++ b/block/blkio.c @@ -678,7 +678,7 @@ static int blkio_virtio_blk_connect(BlockDriverState *bs, QDict *options, const char *path = qdict_get_try_str(options, "path"); BDRVBlkioState *s = bs->opaque; bool fd_supported = false; - int fd, ret; + int fd = -1, ret; if (!path) { error_setg(errp, "missing 'path' option"); @@ -719,6 +719,7 @@ static int blkio_virtio_blk_connect(BlockDriverState *bs, QDict *options, if (ret < 0) { fd_supported = false; qemu_close(fd); + fd = -1; } } } @@ -733,14 +734,18 @@ static int blkio_virtio_blk_connect(BlockDriverState *bs, QDict *options, } ret = blkio_connect(s->blkio); + if (ret < 0 && fd >= 0) { + /* Failed to give the FD to libblkio, close it */ + qemu_close(fd); + fd = -1; + } + /* * If the libblkio driver doesn't support the `fd` property, blkio_connect() * will fail with -EINVAL. So let's try calling blkio_connect() again by * directly setting `path`. */ if (fd_supported && ret == -EINVAL) { - qemu_close(fd); - /* * We need to clear the `fd` property we set previously by setting * it to -1. |