Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

An json.h issue reported in a static code analyzer #3361

Closed
sreerajchundayil opened this issue Mar 1, 2022 · 2 comments
Closed

An json.h issue reported in a static code analyzer #3361

sreerajchundayil opened this issue Mar 1, 2022 · 2 comments
Labels
solution: invalid the issue is not related to the library

Comments

@sreerajchundayil
Copy link

sreerajchundayil commented Mar 1, 2022 •
edited
Loading

I am just attaching one of the 'static code analyzer' reported issue. Please have a look.
nlohmann
Not sure whether it's a false positive.
json version is 3.9.1.
@falbrechtskirchinger
Copy link
Contributor

AFAICT, this is a false positive. json_value is a union type and its destroy(value_t) member function is used to deallocate the active member.

@nlohmann
Copy link
Owner

nlohmann commented Mar 2, 2022

I agree. Thanks for reporting anyway!

@nlohmann nlohmann added solution: invalid the issue is not related to the library and removed kind: bug labels Mar 2, 2022
@nlohmann nlohmann closed this as completed Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
solution: invalid the issue is not related to the library
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nlohmann @falbrechtskirchinger @sreerajchundayil