diff options
| author | Tiwei Bie <tiwei.btw@antgroup.com> | 2025-07-08 17:04:03 +0800 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2025-07-11 08:49:18 +0200 |
| commit | 4c916e3b224a02019b3cc3983a15f32bfd9a22df (patch) | |
| tree | 6f8842503972f7db6d9c6e9be108897c228d579d | |
| parent | 5d2c5b8c410c13995aeee7fdf52b7184a084d506 (diff) | |
um: rtc: Avoid shadowing err in uml_rtc_start()
Remove the declaration of 'err' inside the 'if (timetravel)' block,
as it would otherwise be unavailable outside that block, potentially
leading to uml_rtc_start() returning an uninitialized value.
Fixes: dde8b58d5127 ("um: add a pseudo RTC")
Signed-off-by: Tiwei Bie <tiwei.btw@antgroup.com>
Link: https://patch.msgid.link/20250708090403.1067440-5-tiwei.bie@linux.dev
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
| -rw-r--r-- | arch/um/drivers/rtc_user.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/um/drivers/rtc_user.c b/arch/um/drivers/rtc_user.c index 51e79f3148cd4..67912fcf7b286 100644 --- a/arch/um/drivers/rtc_user.c +++ b/arch/um/drivers/rtc_user.c @@ -28,7 +28,7 @@ int uml_rtc_start(bool timetravel) int err; if (timetravel) { - int err = os_pipe(uml_rtc_irq_fds, 1, 1); + err = os_pipe(uml_rtc_irq_fds, 1, 1); if (err) goto fail; } else { |