diff options
| author | Varun Prakash <varun@chelsio.com> | 2017-07-23 10:33:45 -0400 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2017-07-30 18:32:48 -0400 |
| commit | 310d40a973c560a24c79f84cb5f16dc540a05686 (patch) | |
| tree | c0532c34b8c0a1261aa8cbeed3a6a3b4c1b71ee6 | |
| parent | ea8dc5b4cd2195ee582cae28afa4164c6dea1738 (diff) | |
iscsi-target: fix invalid flags in text response
In case of multiple text responses iscsi-target
sets both 'F' and 'C' bit for the final text response
pdu, this issue happens because hdr->flags is not
zeroed out before ORing with 'F' bit.
This patch removes the | operator to fix this issue.
Signed-off-by: Varun Prakash <varun@chelsio.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
| -rw-r--r-- | drivers/target/iscsi/iscsi_target.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/target/iscsi/iscsi_target.c b/drivers/target/iscsi/iscsi_target.c index 2688918b879a..12803de99400 100644 --- a/drivers/target/iscsi/iscsi_target.c +++ b/drivers/target/iscsi/iscsi_target.c | |||
| @@ -3488,9 +3488,9 @@ iscsit_build_text_rsp(struct iscsi_cmd *cmd, struct iscsi_conn *conn, | |||
| 3488 | return text_length; | 3488 | return text_length; |
| 3489 | 3489 | ||
| 3490 | if (completed) { | 3490 | if (completed) { |
| 3491 | hdr->flags |= ISCSI_FLAG_CMD_FINAL; | 3491 | hdr->flags = ISCSI_FLAG_CMD_FINAL; |
| 3492 | } else { | 3492 | } else { |
| 3493 | hdr->flags |= ISCSI_FLAG_TEXT_CONTINUE; | 3493 | hdr->flags = ISCSI_FLAG_TEXT_CONTINUE; |
| 3494 | cmd->read_data_done += text_length; | 3494 | cmd->read_data_done += text_length; |
| 3495 | if (cmd->targ_xfer_tag == 0xFFFFFFFF) | 3495 | if (cmd->targ_xfer_tag == 0xFFFFFFFF) |
| 3496 | cmd->targ_xfer_tag = session_get_next_ttt(conn->sess); | 3496 | cmd->targ_xfer_tag = session_get_next_ttt(conn->sess); |
