Ping? Markus Armbruster <arm...@redhat.com> writes:
> The transfer length depends on field BYTCHK, which is encoded in byte > 1, bits 1..2. However, the guard for for case BYTCHK=11b doesn't > work, and we get case 01b instead. Fix it. > > Note that since emulated scsi-hd fails the command outright, it takes > SCSI passthrough of a device that actually implements VERIFY with > BYTCHK=11b to make the bug bite. > > Screwed up in commit d12ad44. Spotted by Coverity. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > hw/scsi/scsi-bus.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/scsi/scsi-bus.c b/hw/scsi/scsi-bus.c > index 50b89ad..285a58b 100644 > --- a/hw/scsi/scsi-bus.c > +++ b/hw/scsi/scsi-bus.c > @@ -909,7 +909,7 @@ static int scsi_req_length(SCSICommand *cmd, SCSIDevice > *dev, uint8_t *buf) > case VERIFY_16: > if ((buf[1] & 2) == 0) { > cmd->xfer = 0; > - } else if ((buf[1] & 4) == 1) { > + } else if ((buf[1] & 4) != 0) { > cmd->xfer = 1; > } > cmd->xfer *= dev->blocksize;