]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
ata: libata-core: fix sloppy typing in ata_id_n_sectors()
authorSergey Shtylyov <s.shtylyov@omp.ru>
Fri, 13 May 2022 20:50:14 +0000 (23:50 +0300)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Mon, 13 Jun 2022 02:42:58 +0000 (11:42 +0900)
commit79ad6a561958777d6d8ecfa66e51d09d35ec6450
tree865f89ae992297ebfef43478ac508a0fc1206b43
parent6cd379f75f424b874ea1f35e3c02fa4090246eaf
ata: libata-core: fix sloppy typing in ata_id_n_sectors()

The code multiplying the # of cylinders/heads/sectors in ata_id_n_sectors()
to get a disk capacity implicitly uses the *int* type for that calculation
and casting the result to 'u64' before returning ensues a sign extension.
Explicitly casting the 'u16' typed multipliers to 'u32' results in avoiding
a sign extension instruction and so in a more compact code...

Found by Linux Verification Center (linuxtesting.org) with the SVACE static
analysis tool.

Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/libata-core.c