diff options
| author | Dan Schatzberg <dschatzberg@fb.com> | 2019-11-22 12:00:34 -0800 |
|---|---|---|
| committer | Greg Kurz <groug@kaod.org> | 2019-11-23 15:51:48 +0100 |
| commit | 68d654daee4364a0eca589a547d716084d9cb33d (patch) | |
| tree | 206d0160ee0205abaaa695575cbac7ecb005c82a | |
| parent | 2061735ff09f9d5e67c501a96227b470e7de69b1 (diff) | |
| download | qemu-68d654daee4364a0eca589a547d716084d9cb33d.zip qemu-68d654daee4364a0eca589a547d716084d9cb33d.tar.gz qemu-68d654daee4364a0eca589a547d716084d9cb33d.tar.bz2 | |
9pfs: Fix divide by zero bug
Some filesystems may return 0s in statfs (trivially, a FUSE filesystem
can do so). QEMU should handle this gracefully and just behave the
same as if statfs failed.
Signed-off-by: Dan Schatzberg <dschatzberg@fb.com>
Acked-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
Signed-off-by: Greg Kurz <groug@kaod.org>
| -rw-r--r-- | hw/9pfs/9p.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c index 37abcdb..520177f 100644 --- a/hw/9pfs/9p.c +++ b/hw/9pfs/9p.c @@ -1834,8 +1834,10 @@ static int32_t coroutine_fn get_iounit(V9fsPDU *pdu, V9fsPath *path) * and as well as less than (client msize - P9_IOHDRSZ)) */ if (!v9fs_co_statfs(pdu, path, &stbuf)) { - iounit = stbuf.f_bsize; - iounit *= (s->msize - P9_IOHDRSZ)/stbuf.f_bsize; + if (stbuf.f_bsize) { + iounit = stbuf.f_bsize; + iounit *= (s->msize - P9_IOHDRSZ) / stbuf.f_bsize; + } } if (!iounit) { iounit = s->msize - P9_IOHDRSZ; |