MFC r298669:

r298669 (by cem):

iscsi_initiator(4): Fix use-after-free, double-free

ism_stop() already destroys and frees 'sp', including a call to ic_destroy().
Don't dereference 'sp' after ism_stop() and don't invoke ic_destroy() on the
freed memory either.

CIDs:		1006109, 1304861
This commit is contained in:
ngie 2016-05-13 08:36:33 +00:00
parent 20d9f6b6ed
commit 73cdda036c

View File

@ -807,8 +807,6 @@ iscsi_stop(void)
TAILQ_FOREACH_SAFE(sp, &isc->isc_sess, sp_link, sp_tmp) {
//XXX: check for activity ...
ism_stop(sp);
if(sp->cam_sim != NULL)
ic_destroy(sp);
}
mtx_destroy(&isc->isc_mtx);
sx_destroy(&isc->unit_sx);