Stefan Hajnoczi <stefa...@redhat.com> 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?