Skip to content

Fix #12327 FP memleak when storing pointer in object / #12332 FN memleak for allocation functions with std:: prefix - #5861

Merged
chrchr-github merged 18 commits into
cppcheck-opensource:mainfrom
chrchr-github:chr_Fix12327
Jan 18, 2024
Merged

Fix #12327 FP memleak when storing pointer in object / #12332 FN memleak for allocation functions with std:: prefix#5861
chrchr-github merged 18 commits into
cppcheck-opensource:mainfrom
chrchr-github:chr_Fix12327

Conversation

@chrchr-github

@chrchr-githubchrchr-github commented Jan 8, 2024

Copy link
Copy Markdown
Collaborator

No description provided.

@chrchr-githubchrchr-github changed the title Fix #12327 FP memleak when storing pointer in objectFix #12327 FP memleak when storing pointer in object / Fix #12332 FN memleak for allocation functions with std:: prefixJan 8, 2024
@chrchr-githubchrchr-github changed the title Fix #12327 FP memleak when storing pointer in object / Fix #12332 FN memleak for allocation functions with std:: prefixFix #12327 FP memleak when storing pointer in object / #12332 FN memleak for allocation functions with std:: prefixJan 8, 2024
danmar
danmar reviewed Jan 13, 2024
View reviewed changes
Comment threadlib/library.cpp Outdated
@chrchr-github

chrchr-github commented Jan 15, 2024

Copy link
Copy Markdown
CollaboratorAuthor

Btw: reallocarray() is a nonstandard extension that first appeared in OpenBSD 5.6 and FreeBSD 11.0.

danmar
danmar reviewed Jan 17, 2024
View reviewed changes
Comment threadlib/library.cpp Outdated
danmar
danmar approved these changes Jan 18, 2024
View reviewed changes

@danmardanmar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@chrchr-github
chrchr-github merged commit 303652c into cppcheck-opensource:mainJan 18, 2024
@chrchr-github
chrchr-github deleted the chr_Fix12327 branch January 18, 2024 19:13
@firewavefirewave mentioned this pull request Mar 19, 2024
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danmardanmardanmar approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chrchr-github@danmar