diff options
| author | Swen Schillig <swen@vnet.ibm.com> | 2010-11-17 14:23:42 +0100 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@suse.de> | 2010-12-09 09:41:19 -0600 |
| commit | d3e1088d68735eb7da12f79a0c3c0d951cbc89f1 (patch) | |
| tree | cebed32def6c9a63df52800d1ad63a0366c38f11 /drivers/s390/scsi | |
| parent | 6fbf25e86beef1c6719e760a241a7aef9ad145e3 (diff) | |
| download | kernel_samsung_tuna-d3e1088d68735eb7da12f79a0c3c0d951cbc89f1.zip kernel_samsung_tuna-d3e1088d68735eb7da12f79a0c3c0d951cbc89f1.tar.gz kernel_samsung_tuna-d3e1088d68735eb7da12f79a0c3c0d951cbc89f1.tar.bz2 | |
[SCSI] zfcp: No ERP escalation on gpn_ft eval
If the evaluation of GPN_FT requests wants to remove an invalid port
from the system the zfcp_erp_port_shutdown function is triggered.
Depending on the system status a superior action (e.g. adapter reopen)
is required. This can lead to an invalid mem access of the port struct
which might be freed at the time since the superior action is not
holding a reference of the port which triggered this ERP action.
Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/s390/scsi')
| -rw-r--r-- | drivers/s390/scsi/zfcp_erp.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/s390/scsi/zfcp_erp.c b/drivers/s390/scsi/zfcp_erp.c index d37c733..63422c1 100644 --- a/drivers/s390/scsi/zfcp_erp.c +++ b/drivers/s390/scsi/zfcp_erp.c @@ -156,6 +156,8 @@ static int zfcp_erp_required_act(int want, struct zfcp_adapter *adapter, if (!(a_status & ZFCP_STATUS_COMMON_RUNNING) || a_status & ZFCP_STATUS_COMMON_ERP_FAILED) return 0; + if (p_status & ZFCP_STATUS_COMMON_NOESC) + return need; if (!(a_status & ZFCP_STATUS_COMMON_UNBLOCKED)) need = ZFCP_ERP_ACTION_REOPEN_ADAPTER; /* fall through */ |