diff options
| author | Philippe Mathieu-Daudé <philmd@redhat.com> | 2021-09-02 09:00:18 +0200 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@redhat.com> | 2021-09-07 09:08:24 +0100 |
| commit | 3f4c0affcfbc7c1da0b117bee0cd5be6f52fa3e5 (patch) | |
| tree | 2ffd855beae15267917eda1cffe34ce9d498b965 /util | |
| parent | a990858b0cc0a7fcbf6e0ee102603e6adbbbbaef (diff) | |
| download | qemu-3f4c0affcfbc7c1da0b117bee0cd5be6f52fa3e5.zip qemu-3f4c0affcfbc7c1da0b117bee0cd5be6f52fa3e5.tar.gz qemu-3f4c0affcfbc7c1da0b117bee0cd5be6f52fa3e5.tar.bz2 | |
util/vfio-helpers: Remove unreachable code in qemu_vfio_dma_map()
qemu_vfio_add_mapping() returns a pointer to an indexed entry
in pre-allocated QEMUVFIOState::mappings[], thus can not be NULL.
Remove the pointless check.
Reviewed-by: Klaus Jensen <k.jensen@samsung.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-id: 20210902070025.197072-5-philmd@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'util')
| -rw-r--r-- | util/vfio-helpers.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/util/vfio-helpers.c b/util/vfio-helpers.c index d956866..e790922 100644 --- a/util/vfio-helpers.c +++ b/util/vfio-helpers.c @@ -751,10 +751,6 @@ int qemu_vfio_dma_map(QEMUVFIOState *s, void *host, size_t size, } mapping = qemu_vfio_add_mapping(s, host, size, index + 1, iova0); - if (!mapping) { - ret = -ENOMEM; - goto out; - } assert(qemu_vfio_verify_mappings(s)); ret = qemu_vfio_do_mapping(s, host, size, iova0); if (ret) { |