* [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup() [not found] <CGME20180509001258epcas1p4e8db163d8babd90284dd6afb6a96eb0f@epcas1p4.samsung.com> @ 2018-05-09 0:12 ` Ji-Hun Kim [not found] ` <CGME20180509001300epcas2p3971900d48408aef6a80f59b9480aac16@epcas2p3.samsung.com> ` (2 more replies) 0 siblings, 3 replies; 6+ messages in thread From: Ji-Hun Kim @ 2018-05-09 0:12 UTC (permalink / raw) To: peterhuewe, tweek Cc: jarkko.sakkinen, jgg, arnd, gregkh, javierm, sonnyrao, ji_hun.kim, enric.balletbo, linux-integrity, linux-kernel, kernel-janitors Use kmemdup rather than duplicating its implementation. Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> --- drivers/char/tpm/eventlog/of.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/char/tpm/eventlog/of.c b/drivers/char/tpm/eventlog/of.c index b7cac47..bba5fba 100644 --- a/drivers/char/tpm/eventlog/of.c +++ b/drivers/char/tpm/eventlog/of.c @@ -69,14 +69,12 @@ int tpm_read_log_of(struct tpm_chip *chip) return -EIO; } - log->bios_event_log = kmalloc(size, GFP_KERNEL); + log->bios_event_log = kmemdup(__va(base), size, GFP_KERNEL); if (!log->bios_event_log) return -ENOMEM; log->bios_event_log_end = log->bios_event_log + size; - memcpy(log->bios_event_log, __va(base), size); - if (chip->flags & TPM_CHIP_FLAG_TPM2) return EFI_TCG2_EVENT_LOG_FORMAT_TCG_2; return EFI_TCG2_EVENT_LOG_FORMAT_TCG_1_2; -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <CGME20180509001300epcas2p3971900d48408aef6a80f59b9480aac16@epcas2p3.samsung.com>]
* [PATCH 2/2] tpm: replace kmalloc() + memcpy() with kmemdup() [not found] ` <CGME20180509001300epcas2p3971900d48408aef6a80f59b9480aac16@epcas2p3.samsung.com> @ 2018-05-09 0:12 ` Ji-Hun Kim 2018-05-09 19:16 ` James Morris 2018-05-14 10:58 ` Jarkko Sakkinen 0 siblings, 2 replies; 6+ messages in thread From: Ji-Hun Kim @ 2018-05-09 0:12 UTC (permalink / raw) To: peterhuewe, tweek Cc: jarkko.sakkinen, jgg, arnd, gregkh, javierm, sonnyrao, ji_hun.kim, enric.balletbo, linux-integrity, linux-kernel, kernel-janitors Use kmemdup rather than duplicating its implementation. Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> --- drivers/char/tpm/eventlog/efi.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/char/tpm/eventlog/efi.c b/drivers/char/tpm/eventlog/efi.c index 68f1e7e..3e673ab 100644 --- a/drivers/char/tpm/eventlog/efi.c +++ b/drivers/char/tpm/eventlog/efi.c @@ -51,10 +51,9 @@ int tpm_read_log_efi(struct tpm_chip *chip) } /* malloc EventLog space */ - log->bios_event_log = kmalloc(log_size, GFP_KERNEL); + log->bios_event_log = kmemdup(log_tbl->log, log_size, GFP_KERNEL); if (!log->bios_event_log) goto err_memunmap; - memcpy(log->bios_event_log, log_tbl->log, log_size); log->bios_event_log_end = log->bios_event_log + log_size; tpm_log_version = log_tbl->version; -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] tpm: replace kmalloc() + memcpy() with kmemdup() 2018-05-09 0:12 ` [PATCH 2/2] " Ji-Hun Kim @ 2018-05-09 19:16 ` James Morris 2018-05-14 10:58 ` Jarkko Sakkinen 1 sibling, 0 replies; 6+ messages in thread From: James Morris @ 2018-05-09 19:16 UTC (permalink / raw) To: Ji-Hun Kim Cc: peterhuewe, tweek, jarkko.sakkinen, jgg, arnd, gregkh, javierm, sonnyrao, enric.balletbo, linux-integrity, linux-kernel, kernel-janitors On Wed, 9 May 2018, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> Reviewed-by: James Morris <james.morris@microsoft.com> -- James Morris <jmorris@namei.org> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] tpm: replace kmalloc() + memcpy() with kmemdup() 2018-05-09 0:12 ` [PATCH 2/2] " Ji-Hun Kim 2018-05-09 19:16 ` James Morris @ 2018-05-14 10:58 ` Jarkko Sakkinen 1 sibling, 0 replies; 6+ messages in thread From: Jarkko Sakkinen @ 2018-05-14 10:58 UTC (permalink / raw) To: Ji-Hun Kim Cc: peterhuewe, tweek, jgg, arnd, gregkh, javierm, sonnyrao, enric.balletbo, linux-integrity, linux-kernel, kernel-janitors On Wed, May 09, 2018 at 09:12:37AM +0900, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> Reviewed-by: Jarkko Sakkinen <jarkko.sakkine@linux.intel.com> /Jarkko ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup() 2018-05-09 0:12 ` [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup() Ji-Hun Kim [not found] ` <CGME20180509001300epcas2p3971900d48408aef6a80f59b9480aac16@epcas2p3.samsung.com> @ 2018-05-09 19:16 ` James Morris 2018-05-14 10:58 ` Jarkko Sakkinen 2 siblings, 0 replies; 6+ messages in thread From: James Morris @ 2018-05-09 19:16 UTC (permalink / raw) To: Ji-Hun Kim Cc: peterhuewe, tweek, jarkko.sakkinen, jgg, arnd, gregkh, javierm, sonnyrao, enric.balletbo, linux-integrity, linux-kernel, kernel-janitors On Wed, 9 May 2018, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> Reviewed-by: James Morris <james.morris@microsoft.com> -- James Morris <jmorris@namei.org> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup() 2018-05-09 0:12 ` [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup() Ji-Hun Kim [not found] ` <CGME20180509001300epcas2p3971900d48408aef6a80f59b9480aac16@epcas2p3.samsung.com> 2018-05-09 19:16 ` [PATCH 1/2] " James Morris @ 2018-05-14 10:58 ` Jarkko Sakkinen 2 siblings, 0 replies; 6+ messages in thread From: Jarkko Sakkinen @ 2018-05-14 10:58 UTC (permalink / raw) To: Ji-Hun Kim Cc: peterhuewe, tweek, jgg, arnd, gregkh, javierm, sonnyrao, enric.balletbo, linux-integrity, linux-kernel, kernel-janitors On Wed, May 09, 2018 at 09:12:36AM +0900, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> Reviewed-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> /Jarkko ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-05-14 10:58 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <CGME20180509001258epcas1p4e8db163d8babd90284dd6afb6a96eb0f@epcas1p4.samsung.com> 2018-05-09 0:12 ` [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup() Ji-Hun Kim [not found] ` <CGME20180509001300epcas2p3971900d48408aef6a80f59b9480aac16@epcas2p3.samsung.com> 2018-05-09 0:12 ` [PATCH 2/2] " Ji-Hun Kim 2018-05-09 19:16 ` James Morris 2018-05-14 10:58 ` Jarkko Sakkinen 2018-05-09 19:16 ` [PATCH 1/2] " James Morris 2018-05-14 10:58 ` Jarkko Sakkinen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).