md/drbd drivers use 'signed int' variable to track sync vs non-sync IO, and judge whether sync IO needs to be throttled by signed comparison. If the value of the variable is greater than INT_MAX or close to INT_MAX, some undefined behavior may occur. Fix by using 64bit signed integer type. The v2 "md: Fix undefined behaviour in is_mddev_idle" patch differences to v1: - add ubsan info in message - use 64bit signed integer type instead of long type; - move sync_io variable form struct gendisk to struct md_rdev, and modify md_sync_acct() and md_sync_acct_bio() to fit for this change. Li Jinlin (3): md: Fix undefined behaviour in is_mddev_idle drdb: Fix undefined behaviour in drbd_rs_c_min_rate_throttle drdb: Remove useless variable in struct drbd_device drivers/block/drbd/drbd_bitmap.c | 2 +- drivers/block/drbd/drbd_int.h | 5 ++--- drivers/block/drbd/drbd_main.c | 3 +-- drivers/block/drbd/drbd_receiver.c | 12 ++++++------ drivers/block/drbd/drbd_state.c | 1 - drivers/block/drbd/drbd_worker.c | 5 ++--- drivers/md/md.c | 6 +++--- drivers/md/md.h | 13 +++++++++---- drivers/md/raid1.c | 4 ++-- drivers/md/raid10.c | 24 ++++++++++++------------ drivers/md/raid5.c | 4 ++-- include/linux/genhd.h | 1 - 12 files changed, 40 insertions(+), 40 deletions(-) -- 2.27.0
