diff options
| author | Mikulas Patocka <[email protected]> | 2022-09-14 16:55:51 -0400 |
|---|---|---|
| committer | Jens Axboe <[email protected]> | 2022-09-15 00:25:17 -0600 |
| commit | c4fa368466cc1b60bb92f867741488930ddd6034 (patch) | |
| tree | a80c5a294daf07f3ff4951a5ee205721f8cdb48c /drivers/net/wwan/iosm/iosm_ipc_protocol.c | |
| parent | 56f99b8d06ef1ed1c9730948f9f05ac2b930a20b (diff) | |
blk-lib: fix blkdev_issue_secure_erase
There's a bug in blkdev_issue_secure_erase. The statement
"unsigned int len = min_t(sector_t, nr_sects, max_sectors);"
sets the variable "len" to the length in sectors, but the statement
"bio->bi_iter.bi_size = len" treats it as if it were in bytes.
The statements "sector += len << SECTOR_SHIFT" and "nr_sects -= len <<
SECTOR_SHIFT" are thinko.
This patch fixes it.
Signed-off-by: Mikulas Patocka <[email protected]>
Cc: [email protected] # v5.19
Fixes: 44abff2c0b97 ("block: decouple REQ_OP_SECURE_ERASE from REQ_OP_DISCARD")
Link: https://lore.kernel.org/r/alpine.LRH.2.02.2209141549480.28100@file01.intranet.prod.int.rdu2.redhat.com
Signed-off-by: Jens Axboe <[email protected]>
Diffstat (limited to 'drivers/net/wwan/iosm/iosm_ipc_protocol.c')
0 files changed, 0 insertions, 0 deletions