Skip to content

module-adapter: Add NULL check for sink buffer#10522

Merged
lgirdwood merged 1 commit intothesofproject:mainfrom
wjablon1:sink_null
Feb 5, 2026
Merged

module-adapter: Add NULL check for sink buffer#10522
lgirdwood merged 1 commit intothesofproject:mainfrom
wjablon1:sink_null

Conversation

@wjablon1
Copy link
Contributor

@wjablon1 wjablon1 commented Feb 5, 2026

Just a corner case found in a test with incorrect topology. I think it should be fixed.

Return error if a sink buffer is not present before it is accessed while calculating period size in module_adapter_prepare. This might occur when a pipeline is built incorrectly (without binding) and the sink buffer is not connected to a module.

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch @wjablon1 ! - just the one comment and good to merge.

Copy link
Contributor

@tmleman tmleman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @wjablon1 , great finding!

Return error if a sink buffer is not present before it is accessed while
calculating period size in module_adapter_prepare. This might occur when
a pipeline is built incorrectly (without binding) and the sink buffer is
not connected to a module.

Signed-off-by: Wojciech Jablonski <wojciech.jablonski@intel.com>
Copy link
Collaborator

@softwarecki softwarecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good spot!

@lgirdwood lgirdwood merged commit d065d54 into thesofproject:main Feb 5, 2026
45 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants