The following commit has been merged into the ras/core branch of tip: Commit-ID: 5ba7821bcf7d82e35582fce8fe65cd160a6954b4 Gitweb: https://git.kernel.org/tip/5ba7821bcf7d82e35582fce8fe65cd160a6954b4 Author: Gustavo A. R. Silva <gustavoars@xxxxxxxxxx> AuthorDate: Wed, 17 Jun 2020 16:17:34 -05:00 Committer: Borislav Petkov <bp@xxxxxxx> CommitterDate: Thu, 18 Jun 2020 13:24:23 +02:00 x86/mce/dev-mcelog: Use struct_size() helper in kzalloc() Make use of the struct_size() helper instead of an open-coded version in order to avoid any potential type mistakes. This code was detected with the help of Coccinelle and, audited and fixed manually. Signed-off-by: Gustavo A. R. Silva <gustavoars@xxxxxxxxxx> Signed-off-by: Borislav Petkov <bp@xxxxxxx> Acked-by: Tony Luck <tony.luck@xxxxxxxxx> Link: https://lkml.kernel.org/r/20200617211734.GA9636@embeddedor --- arch/x86/kernel/cpu/mce/dev-mcelog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/cpu/mce/dev-mcelog.c b/arch/x86/kernel/cpu/mce/dev-mcelog.c index 43c4660..03e5105 100644 --- a/arch/x86/kernel/cpu/mce/dev-mcelog.c +++ b/arch/x86/kernel/cpu/mce/dev-mcelog.c @@ -345,7 +345,7 @@ static __init int dev_mcelog_init_device(void) int err; mce_log_len = max(MCE_LOG_MIN_LEN, num_online_cpus()); - mcelog = kzalloc(sizeof(*mcelog) + mce_log_len * sizeof(struct mce), GFP_KERNEL); + mcelog = kzalloc(struct_size(mcelog, entry, mce_log_len), GFP_KERNEL); if (!mcelog) return -ENOMEM;