On Wed, 15 Aug 2018 10:44:34 -0500, Mike Christie <mchristi@xxxxxxxxxx> wrote: > This is the issue I said was fixed in: > > https://www.spinics.net/lists/target-devel/msg17018.html I did apply this patch, yes. It misses the "if(...){kfree(sess); return ret;}" right after the iscsi_login_set_conn_values call, which is what my patch addresses. > Your patch still has issues where the sess_ops can be double freed, > and there is a issue with the idr. I believe you mean the other issues fixed by the patch at above URL. And as I applied that patch, I indeed did not touch these. Regards, -- Vincent Pelletier