diff options
author | Dan Carpenter <error27@gmail.com> | 2010-04-09 22:07:36 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-04-11 14:03:59 -0500 |
commit | ef7d17a907d89d590831a3995ed8638a9ad394bc (patch) | |
tree | 89f3c242e46e157c0ef60f5a4afd5a9a319dcb14 | |
parent | 9f8f3aa640ae5da220eea95215317f19ace91481 (diff) | |
download | op-kernel-dev-ef7d17a907d89d590831a3995ed8638a9ad394bc.zip op-kernel-dev-ef7d17a907d89d590831a3995ed8638a9ad394bc.tar.gz |
[SCSI] iscsi_tcp: change & to &&
This change is basically a clean up.
datadgst_en is an int which comes from the user. I didn't see anything
limiting it to 1 and 0 although obviously that's what it's supposed to
be. If the user passed in 2 this test would fail.
This same if condition is repeated in another function and it uses &&
there.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
-rw-r--r-- | drivers/scsi/libiscsi_tcp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/libiscsi_tcp.c b/drivers/scsi/libiscsi_tcp.c index 5c92620..8eeb39f 100644 --- a/drivers/scsi/libiscsi_tcp.c +++ b/drivers/scsi/libiscsi_tcp.c @@ -421,7 +421,7 @@ iscsi_tcp_data_recv_prep(struct iscsi_tcp_conn *tcp_conn) struct iscsi_conn *conn = tcp_conn->iscsi_conn; struct hash_desc *rx_hash = NULL; - if (conn->datadgst_en & + if (conn->datadgst_en && !(conn->session->tt->caps & CAP_DIGEST_OFFLOAD)) rx_hash = tcp_conn->rx_hash; |