Skip to content

fix: free spectrum buffers on isinf path in SP_Summaries_welch_rect#40

Open
m-muecke wants to merge 1 commit into
DynamicsAndNeuralSystems:mainfrom
m-muecke:fix/sp-summaries-welch-leak
Open

fix: free spectrum buffers on isinf path in SP_Summaries_welch_rect#40
m-muecke wants to merge 1 commit into
DynamicsAndNeuralSystems:mainfrom
m-muecke:fix/sp-summaries-welch-leak

Conversation

@m-muecke

@m-muecke m-muecke commented Jun 15, 2026

Copy link
Copy Markdown

No description provided.

@m-muecke m-muecke force-pushed the fix/sp-summaries-welch-leak branch from 12e4a0d to e31a18e Compare June 15, 2026 08:38
@m-muecke m-muecke changed the title Fix memory leak on isinf path in SP_Summaries_welch_rect fix: free spectrum buffers on isinf path in SP_Summaries_welch_rect Jun 15, 2026
@hendersontrent

Copy link
Copy Markdown
Collaborator

@benfulcher this was also flagged for me in Rcatch22 by the same contributor, and so I have inspected the C code and confirmed the free() calls here are missing and represent a memory leak.

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.

2 participants