OSDN Git Service

scsi: core: Fix scsi_mode_select() buffer length handling
authorDamien Le Moal <damien.lemoal@wdc.com>
Fri, 20 Aug 2021 07:02:54 +0000 (16:02 +0900)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 29 Sep 2021 04:10:57 +0000 (00:10 -0400)
commita7d6840bed0c2b16ac3071b74b5fcf08fc488241
tree7788b3578e71ad8122198d8587b515149485fbc1
parent17b49bcbf8351d3dbe57204468ac34f033ed60bc
scsi: core: Fix scsi_mode_select() buffer length handling

The MODE SELECT(6) command allows handling mode page buffers that are up to
255 bytes, including the 4 byte header needed in front of the page
buffer. For requests larger than this limit, automatically use the MODE
SELECT(10) command.

In both cases, since scsi_mode_select() adds the mode select page header,
checks on the buffer length value must include this header size to avoid
overflows of the command CDB allocation length field.

While at it, use put_unaligned_be16() for setting the header block
descriptor length and CDB allocation length when using MODE SELECT(10).

[mkp: fix MODE SENSE vs. MODE SELECT confusion]

Link: https://lore.kernel.org/r/20210820070255.682775-3-damien.lemoal@wdc.com
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/scsi_lib.c