EDAC/versal: Check user-supplied data before injecting an error
authorShubhrajyoti Datta <shubhrajyoti.datta@amd.com>
Thu, 25 Apr 2024 12:19:41 +0000 (17:49 +0530)
committerBorislav Petkov (AMD) <bp@alien8.de>
Thu, 25 Apr 2024 16:04:47 +0000 (18:04 +0200)
commitde87ba848d5e4c861b7357dd7a91698aed7a5a18
tree5424bf069743025200c00af331b839923a6780c2
parentedbe59428eb0da09958769326a6566d4c9242ae7
EDAC/versal: Check user-supplied data before injecting an error

The function inject_data_ue_store() lacks a NULL check for the user
passed values. To prevent below kernel crash include a NULL check.

Call trace:

  kstrtoull
  kstrtou8
  inject_data_ue_store
  full_proxy_write
  vfs_write
  ksys_write
  __arm64_sys_write
  invoke_syscall
  el0_svc_common.constprop.0
  do_el0_svc
  el0_svc
  el0t_64_sync_handler
  el0t_64_sync

Fixes: 83bf24051a60 ("EDAC/versal: Make the bit position of injected errors configurable")
Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@amd.com>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Link: https://lore.kernel.org/r/20240425121942.26378-3-shubhrajyoti.datta@amd.com
drivers/edac/versal_edac.c