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.

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

vdk-control-service: fix null dereferences #1512

Merged
merged 9 commits into from
Jan 17, 2023

Conversation

dakodakov
Copy link
Collaborator

Why?
Fix null dereference uncovered by coverity scaner

What?
Multiple changes mostly related to json conversions.

What type of change are you making?
Bug fix

Signed-off-by: Dako Dakov ddakov@vmware.com

Why?
Fix null dereference uncovered by coverity scaner

What?
Multiple changes mostly related to json conversions.

What type of change are you making?
Bug fix

Signed-off-by: Dako Dakov ddakov@vmware.com
github-actions and others added 2 commits January 13, 2023 13:45
re-run tests

Signed-off-by: Dako Dakov ddakov@vmware.com
@murphp15
Copy link
Collaborator

murphp15 commented Jan 15, 2023

Hey some of these are good finds.
How do you run coverity?
where do you see the output from it?
Would you be able to include links to it in this PR.

@dakodakov dakodakov merged commit 3e17387 into main Jan 17, 2023
@dakodakov dakodakov deleted the person/ddakov/coverity-string-dereference branch January 17, 2023 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants