Skip to content

ipc4: fix an "unused variable" warning, found by a fuzzer #10001

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

Closed
lyakh wants to merge 2 commits into from
Closed

ipc4: fix an "unused variable" warning, found by a fuzzer #10001

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d9ebdb0
ipc4: fix an "unused variable" warning, found by a fuzzer
lyakh May 12, 2025
746f757
ipc: fix two unused variables in fuzz_isr()
lyakh May 13, 2025
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
2 changes: 1 addition & 1 deletion src/ipc/ipc4/helper.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -942,7 +942,7 @@ int ipc4_process_on_core(uint32_t core, bool blocking)

__cold static const struct comp_driver *ipc4_get_drv(const void *uuid)
{
const struct sof_uuid *const sof_uuid = (const struct sof_uuid *)uuid;
const struct sof_uuid *const sof_uuid __maybe_unused = (const struct sof_uuid *)uuid;
struct comp_driver_list *drivers = comp_drivers_get();
struct list_item *clist;
const struct comp_driver *drv = NULL;
Expand Down
5 changes: 3 additions & 2 deletions src/platform/posix/ipc.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,6 @@ static size_t fuzz_in_sz;
static void fuzz_isr(const void *arg)
{
size_t rem, i, n = MIN(posix_fuzz_sz, sizeof(fuzz_in) - fuzz_in_sz);
bool comp_new = false;
int comp_idx = 0;

for (i = 0; i < n; i++)
fuzz_in[fuzz_in_sz++] = posix_fuzz_buf[i];
Expand All @@ -68,6 +66,9 @@ static void fuzz_isr(const void *arg)
fuzz_in_sz = rem;

#ifdef CONFIG_IPC_MAJOR_3
bool comp_new = false;
int comp_idx = 0;

// One special case: a first byte of 0xff (which is in the
// otherwise-ignored size value at the front of the command --
// we rewrite those) is interpreted as a "component new"
Expand Down
Loading