|
From: | Markus Armbruster |
Subject: | Re: [Qemu-devel] [PATCH 3/3] virtio-blk: Treat read/write beyond end as invalid |
Date: | Mon, 23 Jun 2014 10:47:47 +0200 |
User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Stefan Hajnoczi <address@hidden> writes: > On Thu, Jun 05, 2014 at 02:15:36PM +0200, Markus Armbruster wrote: >> + if (sector > total_sectors || nb_sectors > total_sectors - sector) { >> + return false; >> + } > > if (sector >= total_sectors || ...) { I suspect reading bdrv_check_byte_request() put the '>' in my brain: if ((offset > len) || (len - offset < size)) return -EIO; Don't we need offset >= len here?
[Prev in Thread] | Current Thread | [Next in Thread] |