From f6df62699fac991ac74dbae708206fdbf1799fa2 Mon Sep 17 00:00:00 2001 From: Adrian Bonislawski Date: Mon, 8 Jun 2026 20:57:54 +0200 Subject: [PATCH] audio: init align to avoid uninitialized read in error log sof_dma_get_attribute() can return an error without writing its output parameter, so logging align on that path read an uninitialized value. Initialize align to 0 in the host/dai zephyr and legacy params paths. Signed-off-by: Adrian Bonislawski --- src/audio/dai-legacy.c | 2 +- src/audio/dai-zephyr.c | 2 +- src/audio/host-legacy.c | 2 +- src/audio/host-zephyr.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/audio/dai-legacy.c b/src/audio/dai-legacy.c index 655fa94afb86..fb095a16a626 100644 --- a/src/audio/dai-legacy.c +++ b/src/audio/dai-legacy.c @@ -467,7 +467,7 @@ int dai_common_params(struct dai_data *dd, struct comp_dev *dev, uint32_t period_bytes; uint32_t buffer_size; uint32_t addr_align; - uint32_t align; + uint32_t align = 0; int err; comp_dbg(dev, "dai_params()"); diff --git a/src/audio/dai-zephyr.c b/src/audio/dai-zephyr.c index e49d3e0f0122..fc2a065741cd 100644 --- a/src/audio/dai-zephyr.c +++ b/src/audio/dai-zephyr.c @@ -977,7 +977,7 @@ static int dai_set_dma_buffer(struct dai_data *dd, struct comp_dev *dev, uint32_t buffer_size; uint32_t buffer_size_preferred; uint32_t addr_align; - uint32_t align; + uint32_t align = 0; int err; comp_dbg(dev, "entry"); diff --git a/src/audio/host-legacy.c b/src/audio/host-legacy.c index 3d62e271f518..033c4cf2a09e 100644 --- a/src/audio/host-legacy.c +++ b/src/audio/host-legacy.c @@ -686,7 +686,7 @@ int host_common_params(struct host_data *hd, struct comp_dev *dev, uint32_t period_bytes; uint32_t buffer_size; uint32_t addr_align; - uint32_t align; + uint32_t align = 0; int err; /* host params always installed by pipeline IPC */ diff --git a/src/audio/host-zephyr.c b/src/audio/host-zephyr.c index 90b0821caf5a..e3f79419b389 100644 --- a/src/audio/host-zephyr.c +++ b/src/audio/host-zephyr.c @@ -887,7 +887,7 @@ int host_common_params(struct host_data *hd, struct comp_dev *dev, uint32_t buffer_size; uint32_t buffer_size_preferred; uint32_t addr_align; - uint32_t align; + uint32_t align = 0; int i, err; bool is_scheduling_source = dev == dev->pipeline->sched_comp; uint32_t round_up_size;