From 59f352a819056b46f3aff4c31c9ca212cefc723f Mon Sep 17 00:00:00 2001 From: Sagar Gulab Patil Date: Tue, 19 Jul 2022 06:44:19 +0000 Subject: [PATCH] tegra: cvnas: Fix Cert-C error handling bug In function nvcvnas_power_off casting nvcvsram_readl from unsigned int to int without checking its value may result in lost or misinterpreted data. Fix this by changing data type of val to u32. CID 427418 Bug 3512545 Change-Id: I8be65b1ca73270a4a0d76380207078da745c7057 Signed-off-by: Sagar Gulab Patil Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2747340 Reviewed-by: Sachin Nikam --- drivers/platform/tegra/cvnas/cvnas.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/platform/tegra/cvnas/cvnas.c b/drivers/platform/tegra/cvnas/cvnas.c index 911dadbc..30ec99b7 100644 --- a/drivers/platform/tegra/cvnas/cvnas.c +++ b/drivers/platform/tegra/cvnas/cvnas.c @@ -408,7 +408,7 @@ err_enable_clk: static int nvcvnas_power_off(struct cvnas_device *cvnas_dev) { - int val, i, j; + u32 val, i, j; u32 fcm_pg_seq[] = {0x80, 0xC0, 0xE0, 0xF0, 0xF8, 0xFC, 0xFE, 0xFF};