Skip to content

fixed #13719 - fixed nullptr dereference in Library::detectSmartPointer()#7379

Merged
chrchr-github merged 1 commit intocppcheck-opensource:mainfrom
firewave:smart-null
Mar 19, 2025
Merged

fixed #13719 - fixed nullptr dereference in Library::detectSmartPointer()#7379
chrchr-github merged 1 commit intocppcheck-opensource:mainfrom
firewave:smart-null

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

@firewave
Copy link
Copy Markdown
Collaborator Author

@chrchr-github I am not sure where to place the test from the ticket.

I cannot be bothered right now to provide the full test coverage for that function right now. Will do in a follow.

@chrchr-github
Copy link
Copy Markdown
Collaborator

TestSymbolDatabase should be fine. The test case can also be slightly simplified, as the second constructor is not needed.

@firewave firewave force-pushed the smart-null branch 2 times, most recently from 0b2d529 to af0c5aa Compare March 19, 2025 07:06
@firewave
Copy link
Copy Markdown
Collaborator Author

TestSymbolDatabase should be fine. The test case can also be slightly simplified, as the second constructor is not needed.

Thanks. Reduced it even way more.

@firewave firewave marked this pull request as ready for review March 19, 2025 07:59
@chrchr-github chrchr-github merged commit 0b390be into cppcheck-opensource:main Mar 19, 2025
60 of 61 checks passed
@firewave firewave deleted the smart-null branch March 19, 2025 09:40
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