-----Original Message-----
From: Gustavo A. R. Silva [mailto:[email protected]]
Sent: Thursday, January 3, 2019 10:48 AM
To: Mahesh Rajashekhara <[email protected]>; Don Brace
<[email protected]>; James E.J. Bottomley <[email protected]>; Martin K.
Petersen <[email protected]>
Cc: [email protected]; [email protected];
[email protected]; Gustavo A. R. Silva <[email protected]>
Subject: [PATCH] scsi: smartpqi_init: fix boolean expression in
pqi_device_remove_start
EXTERNAL EMAIL
Fix boolean expression by using logical AND operator '&&'
instead of bitwise operator '&'.
This issue was detected with the help of Coccinelle.
Fixes: 1e46731efd9c ("scsi: smartpqi: check for null device pointers")
Signed-off-by: Gustavo A. R. Silva <[email protected]>
Thanks for the update.
Acked-by: Don Brace <[email protected]>
---
drivers/scsi/smartpqi/smartpqi_init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/smartpqi/smartpqi_init.c
b/drivers/scsi/smartpqi/smartpqi_init.c
index e2fa3f476227..40f58238ce4a 100644
--- a/drivers/scsi/smartpqi/smartpqi_init.c
+++ b/drivers/scsi/smartpqi/smartpqi_init.c
@@ -323,7 +323,7 @@ static inline void pqi_device_remove_start(struct
pqi_scsi_dev *device) static inline bool pqi_device_in_remove(struct
pqi_ctrl_info *ctrl_info,
struct pqi_scsi_dev *device) {
- return device->in_remove & !ctrl_info->in_shutdown;
+ return device->in_remove && !ctrl_info->in_shutdown;
}
static inline void pqi_schedule_rescan_worker_with_delay(
--
2.20.1
