Re: [PATCH] target: Fix a double put in transport_free_session

2021-03-26 Thread Mike Christie
On 3/26/21 7:31 AM, Maurizio Lombardi wrote: > > > Dne 23. 03. 21 v 3:58 Lv Yunlong napsal(a): >> In transport_free_session, se_nacl is got from se_sess >> with the initial reference. If se_nacl->acl_sess_list is >> empty, se_nacl->dynamic_stop is set to true. Then the first >> target_put_nacl(se

Re: [PATCH] target: Fix a double put in transport_free_session

2021-03-26 Thread Maurizio Lombardi
Dne 23. 03. 21 v 3:58 Lv Yunlong napsal(a): > In transport_free_session, se_nacl is got from se_sess > with the initial reference. If se_nacl->acl_sess_list is > empty, se_nacl->dynamic_stop is set to true. Then the first > target_put_nacl(se_nacl) will drop the initial reference > and free se_n

Re: Re: [PATCH] target: Fix a double put in transport_free_session

2021-03-26 Thread lyl2019
; >> 收件人: "Lv Yunlong" , martin.peter...@oracle.com > >> 抄送: linux-s...@vger.kernel.org, target-de...@vger.kernel.org, > >> linux-kernel@vger.kernel.org > >> 主题: Re: [PATCH] target: Fix a double put in transport_free_session > >> > >> On

Re: [PATCH] target: Fix a double put in transport_free_session

2021-03-25 Thread Mike Christie
vger.kernel.org, >> linux-kernel@vger.kernel.org >> 主题: Re: [PATCH] target: Fix a double put in transport_free_session >> >> On 3/22/21 9:58 PM, Lv Yunlong wrote: >>> In transport_free_session, se_nacl is got from se_sess >>> with the initial reference. I

Re: Re: [PATCH] target: Fix a double put in transport_free_session

2021-03-25 Thread lyl2019
> -原始邮件- > 发件人: michael.chris...@oracle.com > 发送时间: 2021-03-24 00:28:35 (星期三) > 收件人: "Lv Yunlong" , martin.peter...@oracle.com > 抄送: linux-s...@vger.kernel.org, target-de...@vger.kernel.org, > linux-kernel@vger.kernel.org > 主题: Re: [PAT

Re: [PATCH] target: Fix a double put in transport_free_session

2021-03-23 Thread michael . christie
On 3/22/21 9:58 PM, Lv Yunlong wrote: > In transport_free_session, se_nacl is got from se_sess > with the initial reference. If se_nacl->acl_sess_list is > empty, se_nacl->dynamic_stop is set to true. Then the first > target_put_nacl(se_nacl) will drop the initial reference > and free se_nacl. Late

[PATCH] target: Fix a double put in transport_free_session

2021-03-22 Thread Lv Yunlong
In transport_free_session, se_nacl is got from se_sess with the initial reference. If se_nacl->acl_sess_list is empty, se_nacl->dynamic_stop is set to true. Then the first target_put_nacl(se_nacl) will drop the initial reference and free se_nacl. Later there is a second target_put_nacl() to put se_