Skip to content

Audio modules fixes #10752

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
lgirdwood wants to merge 3 commits into
base: main
Choose a base branch
from
+38 −11
Open

Audio modules fixes #10752

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
7a514af
audio: asrc: Fix heap double-free during teardown
lrgirdwo May 6, 2026
2080969
audio: mfcc: Add frame_shift validation to prevent division by zero
lrgirdwo May 6, 2026
1ebfd38
audio: mic_privacy_manager: Fix NULL pointer dereference in ZTest
lrgirdwo May 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 26 additions & 11 deletions src/audio/asrc/asrc.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -315,15 +315,25 @@ static void asrc_release_buffers(struct processing_module *mod, struct asrc_farr
static int asrc_free(struct processing_module *mod)
{
struct comp_data *cd = module_get_private_data(mod);
struct comp_dev *dev = mod->dev;

comp_dbg(dev, "entry");
comp_info(mod->dev, "asrc_free() entry");

if (!cd)
return 0;

mod_free(mod, cd->buf);
asrc_release_buffers(mod, cd->asrc_obj);
asrc_free_polyphase_filter(mod, cd->asrc_obj);
mod_free(mod, cd->asrc_obj);
cd->buf = NULL;

if (cd->asrc_obj) {
asrc_release_buffers(mod, cd->asrc_obj);
asrc_free_polyphase_filter(mod, cd->asrc_obj);
mod_free(mod, cd->asrc_obj);
cd->asrc_obj = NULL;
}

mod_free(mod, cd);
module_set_private_data(mod, NULL);

return 0;
}

Expand Down Expand Up @@ -862,12 +872,17 @@ static int asrc_reset(struct processing_module *mod)
asrc_dai_stop_timestamp(cd);

/* Free the allocations those were done in prepare() */
asrc_release_buffers(mod, cd->asrc_obj);
asrc_free_polyphase_filter(mod, cd->asrc_obj);
mod_free(mod, cd->asrc_obj);
mod_free(mod, cd->buf);
cd->asrc_obj = NULL;
cd->buf = NULL;
if (cd->asrc_obj) {
asrc_release_buffers(mod, cd->asrc_obj);
asrc_free_polyphase_filter(mod, cd->asrc_obj);
mod_free(mod, cd->asrc_obj);
cd->asrc_obj = NULL;
}

if (cd->buf) {
mod_free(mod, cd->buf);
cd->buf = NULL;
}

return 0;
}
Expand Down
4 changes: 4 additions & 0 deletions src/audio/mfcc/mfcc_setup.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,10 @@ int mfcc_setup(struct processing_module *mod, int max_frames, int sample_rate, i
state->emph.coef = -config->preemphasis_coefficient; /* Negate config parameter */
fft->fft_size = config->frame_length;
fft->fft_padded_size = 1 << (31 - norm_int32(fft->fft_size)); /* Round up to nearest 2^N */
if (config->frame_shift <= 0) {
comp_err(dev, "frame_shift must be positive");
return -EINVAL;
}
Comment on lines 156 to +161
fft->fft_hop_size = config->frame_shift;
fft->half_fft_size = (fft->fft_padded_size >> 1) + 1;

Expand Down
8 changes: 8 additions & 0 deletions src/audio/mic_privacy_manager/mic_privacy_manager_intel.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,11 @@ int mic_privacy_manager_init(void)
if (!mic_priv_dev)
return -EINVAL;

if (!device_is_ready(mic_priv_dev)) {
LOG_ERR("mic_privacy device not ready");
return -ENODEV;
}
Comment on lines +98 to +101

mic_privacy_api = (struct mic_privacy_api_funcs *)mic_priv_dev->api;
mic_privacy_policy = mic_privacy_api->get_policy();

Expand All @@ -109,6 +114,9 @@ int mic_privacy_manager_init(void)

int mic_privacy_manager_get_policy(void)
{
if (!mic_priv_dev)
return MIC_PRIVACY_HW_MANAGED;

mic_privacy_api = (struct mic_privacy_api_funcs *)mic_priv_dev->api;

return mic_privacy_api->get_policy();
Expand Down
Loading