Il 06/06/2014 15:16, Stefan Hajnoczi ha scritto:
> - req->in = (void *)req->elem->in_sg[req->elem->in_num - 1].iov_base; > + assert(in_iov[in_num - 1].iov_len >= > + sizeof(struct virtio_blk_inhdr));Why use assert() when the rest of the function uses error_report() + exit(1)? Please keep the code consistent.
Sorry, that's my fault. I suggested assert thinking that zero-length iovecs wouldn't be possible here.
Paolo
