Skip to content

fix: double-free in FireFront cleanup#145

Merged
antonio-leblanc merged 1 commit into
forefireAPI:devfrom
yanissrairi:fix/firefront-delete-u
May 12, 2026
Merged

fix: double-free in FireFront cleanup#145
antonio-leblanc merged 1 commit into
forefireAPI:devfrom
yanissrairi:fix/firefront-delete-u

Conversation

@yanissrairi
Copy link
Copy Markdown

src/FireFront.cpp:317 guards u but delete [] targets z. Surrounding lines all follow if (X != 0) delete [] X;, so this looks like a copy-paste typo. With both pointers non-null it leaks u and double-frees z.

One-character fix. Tests still pass locally.

@antonio-leblanc antonio-leblanc merged commit cf0631a into forefireAPI:dev May 12, 2026
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