OSDN Git Service

scsi: target: tcmu: Fix crash on ARM during cmd completion
authorBodo Stroesser <bstroesser@ts.fujitsu.com>
Mon, 29 Jun 2020 09:37:56 +0000 (11:37 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 30 Jun 2020 01:44:57 +0000 (21:44 -0400)
commit5a0c256d96f020e4771f6fd5524b80f89a2d3132
tree084dca3414068ad8d4794286eb5e21d3113bb983
parentb7a80dac0f1f16ca5714b89ace8a3fc079686875
scsi: target: tcmu: Fix crash on ARM during cmd completion

If tcmu_handle_completions() has to process a padding shorter than
sizeof(struct tcmu_cmd_entry), the current call to
tcmu_flush_dcache_range() with sizeof(struct tcmu_cmd_entry) as length
param is wrong and causes crashes on e.g. ARM, because
tcmu_flush_dcache_range() in this case calls
flush_dcache_page(vmalloc_to_page(start)); with start being an invalid
address above the end of the vmalloc'ed area.

The fix is to use the minimum of remaining ring space and sizeof(struct
tcmu_cmd_entry) as the length param.

The patch was tested on kernel 4.19.118.

See https://bugzilla.kernel.org/show_bug.cgi?id=208045#c10

Link: https://lore.kernel.org/r/20200629093756.8947-1-bstroesser@ts.fujitsu.com
Tested-by: JiangYu <lnsyyj@hotmail.com>
Acked-by: Mike Christie <michael.christie@oracle.com>
Signed-off-by: Bodo Stroesser <bstroesser@ts.fujitsu.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/target/target_core_user.c