diff options
author | Avi Kivity <avi@redhat.com> | 2010-05-16 14:59:57 +0300 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2010-05-21 11:49:18 +0200 |
commit | ab89cba12bcf328fd9668b03872c6964122194de (patch) | |
tree | bf012c66eddd97a04dbe4d6795cd10a1ccbba4cc /aio.c | |
parent | 9925227037e77456cccb3230a542f2835bfb6175 (diff) | |
download | qemu-ab89cba12bcf328fd9668b03872c6964122194de.tar.gz qemu-ab89cba12bcf328fd9668b03872c6964122194de.tar.bz2 qemu-ab89cba12bcf328fd9668b03872c6964122194de.zip |
block: fix aio_flush segfaults for read-only protocols (e.g. curl)
Not all block format drivers expose an io_flush method (reasonable for
read-only protocols), so calling io_flush there will immediately segfault.
Fix by checking for the method's existence before calling it.
Signed-off-by: Avi Kivity <avi@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'aio.c')
-rw-r--r-- | aio.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -113,7 +113,9 @@ void qemu_aio_flush(void) qemu_aio_wait(); QLIST_FOREACH(node, &aio_handlers, node) { - ret |= node->io_flush(node->opaque); + if (node->io_flush) { + ret |= node->io_flush(node->opaque); + } } } while (qemu_bh_poll() || ret > 0); } |