mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-30 13:23:02 -04:00
scsi: scsi_debug: Negate wrapped memcmp() result
comp_write_worker() returns true when the compared data matches.
memcmp() returns zero for equal data and non-zero for different data, so
its result must be negated before it is stored in a bool.
The first segment already uses !memcmp(), but the wrapped segment uses
memcmp() directly, reversing the match result. Use !memcmp() there as
well.
Fixes: 38d5c8336e ("scsi_debug: add Report supported opcodes+tmfs; Compare and write")
Cc: stable@vger.kernel.org
Signed-off-by: Xu Rao <raoxu@uniontech.com>
Reviewed-by: John Garry <john.g.garry@oracle.com>
Link: https://patch.msgid.link/E5AD42E9C0E18633+20260803095328.3445311-1-raoxu@uniontech.com
Signed-off-by: Martin K. Petersen (Oracle) <mkp@kernel.org>
This commit is contained in:
committed by
Martin K. Petersen (Oracle)
parent
732cb6bb37
commit
c4f6916a99
@@ -4310,8 +4310,8 @@ static bool comp_write_worker(struct sdeb_store_info *sip, u64 lba, u32 num,
|
||||
if (!res)
|
||||
return res;
|
||||
if (rest)
|
||||
res = memcmp(fsp, arr + ((num - rest) * lb_size),
|
||||
rest * lb_size);
|
||||
res = !memcmp(fsp, arr + ((num - rest) * lb_size),
|
||||
rest * lb_size);
|
||||
if (!res)
|
||||
return res;
|
||||
if (compare_only)
|
||||
|
||||
Reference in New Issue
Block a user