mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-06 02:50:49 +09:00
platform/x86: think-lmi: Use kfree_sensitive instead of kfree
[ Upstream commit 1da0893aed ]
key might contain private part of the key, so better use
kfree_sensitive to free it.
Signed-off-by: Wang Ming <machel@vivo.com>
Link: https://lore.kernel.org/r/20230717101114.18966-1-machel@vivo.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
dea41980d7
commit
aeee50c152
@@ -719,12 +719,12 @@ static ssize_t cert_to_password_store(struct kobject *kobj,
|
|||||||
/* Format: 'Password,Signature' */
|
/* Format: 'Password,Signature' */
|
||||||
auth_str = kasprintf(GFP_KERNEL, "%s,%s", passwd, setting->signature);
|
auth_str = kasprintf(GFP_KERNEL, "%s,%s", passwd, setting->signature);
|
||||||
if (!auth_str) {
|
if (!auth_str) {
|
||||||
kfree(passwd);
|
kfree_sensitive(passwd);
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
}
|
}
|
||||||
ret = tlmi_simple_call(LENOVO_CERT_TO_PASSWORD_GUID, auth_str);
|
ret = tlmi_simple_call(LENOVO_CERT_TO_PASSWORD_GUID, auth_str);
|
||||||
kfree(auth_str);
|
kfree(auth_str);
|
||||||
kfree(passwd);
|
kfree_sensitive(passwd);
|
||||||
|
|
||||||
return ret ?: count;
|
return ret ?: count;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user