Re: [PATCH] scsi: use kzfree to free password

7 views
Skip to first unread message

Lee Duncan

unread,
Jul 28, 2019, 4:59:06 PM7/28/19
to Wang Xiayang, open-...@googlegroups.com, cle...@redhat.com
On 7/27/19 3:39 AM, Wang Xiayang wrote:

> Similar to commit 34bca9bbe7a8 ("cifs: Use kzfree() to free password"),
> password strings allocated for CHAP should be zero-ed before freed.
>
> Signed-off-by: Wang Xiayang <xywan...@sjtu.edu.cn>
> ---
> drivers/scsi/libiscsi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c
> index ebd47c0cf9e9..67d66e6dbdb0 100644
> --- a/drivers/scsi/libiscsi.c
> +++ b/drivers/scsi/libiscsi.c
> @@ -2854,8 +2854,8 @@ void iscsi_session_teardown(struct iscsi_cls_session *cls_session)
>
> iscsi_remove_session(cls_session);
>
> - kfree(session->password);
> - kfree(session->password_in);
> + kzfree(session->password);
> + kzfree(session->password_in);
> kfree(session->username);
> kfree(session->username_in);
> kfree(session->targetname);

Reviewed-by: Lee Duncan <ldu...@suse.com>

Wang Xiayang

unread,
Jul 28, 2019, 5:09:39 PM7/28/19
to ldu...@suse.com, cle...@redhat.com, open-...@googlegroups.com, Wang Xiayang
--
2.11.0

Reply all
Reply to author
Forward
0 new messages