Re: [PATCH 6/6] libata: switch to dynamic allocation insted of ata_scsi_rbuf

Christoph Hellwig
2017-01-10T14:40:56Z

> struct scsi_cmnd *scmd = qc->scsicmd;
> struct scsi_device *sdp = scmd->device;
> - size_t len = sdp->sector_size;
> + ssize_t len = sdp->sector_size;

Julia Lawall's magic checker tool pointed out that len doesn't have
to change to ssize_t here, but the size variable should instead so that
we can properly handle the ENOMEM case below.

I'll wait for some more feedback on the patches and will eventually
resend with that fix.