On 8/15/25 8:58 PM, Tom Rini wrote:
On Mon, Aug 04, 2025 at 07:46:30PM -0700, alif.zakuan.yusla...@altera.com wrote:
From: Alif Zakuan Yuslaimi <alif.zakuan.yusla...@altera.com>
Initialize the actread variable to prevent undefined behavior
that can occur if the variable is used before being assigned a
value.
This will help to prevent potential issues, especially if
actread is used (e.g., read, incremented, or returned) before
being explicitly set elsewhere in the code.
Signed-off-by: Alif Zakuan Yuslaimi <alif.zakuan.yusla...@altera.com>
---
drivers/misc/fs_loader.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/misc/fs_loader.c b/drivers/misc/fs_loader.c
index 66803f4b997..cc07bfa7f14 100644
--- a/drivers/misc/fs_loader.c
+++ b/drivers/misc/fs_loader.c
@@ -1,6 +1,7 @@
// SPDX-License-Identifier: GPL-2.0
/*
* Copyright (C) 2018-2019 Intel Corporation <www.intel.com>
+ * Copyright (C) 2025 Altera Corporation <www.altera.com>
*
*/
@@ -148,7 +149,7 @@ static int _request_firmware_prepare(struct udevice *dev,
*/
static int fw_get_filesystem_firmware(struct udevice *dev)
{
- loff_t actread;
+ loff_t actread = 0;
char *storage_interface, *dev_part, *ubi_mtdpart, *ubi_volume;
int ret;
A one line "initialize a variable" is not sufficient for adding a
Copyright line, please drop in v2, thanks.
Technically they already have a copyright line there, they only updated
the year. This was probably written when altera was intel PSG, now
altera is altera again, so there are two copyright lines, but it is the
same group of people.
