diff options
author | Mike Christie <michaelc@cs.wisc.edu> | 2010-06-09 03:30:08 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-07-27 12:01:39 -0500 |
commit | fa95d206e4a4fb549bdb9fe71091417f4912178f (patch) | |
tree | 1d4adc8b1e38b5b9d0dc836d00c8322631f426eb /drivers/scsi/ipr.h | |
parent | 2cae179486a356aca2a2617f0399f04c66598d8b (diff) | |
download | lwn-fa95d206e4a4fb549bdb9fe71091417f4912178f.tar.gz lwn-fa95d206e4a4fb549bdb9fe71091417f4912178f.zip |
[SCSI] be2iscsi: fix disconnection cleanup
This patch fixes 4 bugs in the connection connect/disconnect
cleanup path.
1. If beiscsi_open_conn fails beiscsi_free_ep was always being
called, and if beiscsi_open_conn failed because beiscsi_get_cid
failed then we would free an unallocated cid.
2. If beiscsi_ep_connect failed due to a beiscsi_open_conn failure
it was leaking iscsi_endpoints.
3. beiscsi_ep_disconnect was leaking iscsi_endpoints.
beiscsi_ep_disconnect should free the iscsi_endpoint. We cannot
do it in beiscsi_conn_stop because that is only called for
iscsi connection cleanup. If beiscsi_ep_connect returns
success, but then the poll function fails or the connect
times out then beiscsi_ep_disconnect will be called to clean
up the ep. The conn_stop callout will not be called in that path.
4. beiscsi_conn_stop was freeing the iscsi_endpoint then accessing
it a couple lines later.
Signed-off-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/ipr.h')
0 files changed, 0 insertions, 0 deletions