Skip to content

Commit 2702f55

Browse files
johnpgarrykawasaki
authored andcommitted
block: avoid possible overflow for chunk_sectors check in blk_stack_limits()
In blk_stack_limits(), we check that the t->chunk_sectors value is a multiple of the t->physical_block_size value. However, by finding the chunk_sectors value in bytes, we may overflow the unsigned int which holds chunk_sectors, so change the check to be based on sectors. Signed-off-by: John Garry <[email protected]> Reviewed-by: Hannes Reinecke <[email protected]>
1 parent 87bbbbc commit 2702f55

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

block/blk-settings.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -779,7 +779,7 @@ int blk_stack_limits(struct queue_limits *t, struct queue_limits *b,
779779
}
780780

781781
/* chunk_sectors a multiple of the physical block size? */
782-
if ((t->chunk_sectors << 9) & (t->physical_block_size - 1)) {
782+
if (t->chunk_sectors % (t->physical_block_size >> SECTOR_SHIFT)) {
783783
t->chunk_sectors = 0;
784784
t->flags |= BLK_FLAG_MISALIGNED;
785785
ret = -1;

0 commit comments

Comments
 (0)