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

Fixed coverity issues in acl_hal_mmd.cpp #229

Merged
merged 1 commit into from
Jan 10, 2023
Merged

Fixed coverity issues in acl_hal_mmd.cpp #229

merged 1 commit into from
Jan 10, 2023

Conversation

haoxian2
Copy link
Contributor

@haoxian2 haoxian2 commented Dec 12, 2022 •
edited
Loading

Fixed coverity issues in acl_hal_mmd.cpp: Type: Resource leak (RESOURCE_LEAK)

This PR is for fixing one of the problems with acl_hal_mmd.cpp.
The other problem will need some refactoring, linked PR: #227

@haoxian2 haoxian2 mentioned this pull request Dec 12, 2022
Closed
@pcolberg pcolberg added this to the 2023.2 milestone Dec 23, 2022
@pcolberg pcolberg added the bug Something isn't working label Dec 23, 2022
zibaiwan
zibaiwan approved these changes Jan 10, 2023
View reviewed changes
Copy link
Contributor

@zibaiwan zibaiwan left a comment

Choose a reason for hiding this comment

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

Thanks @haoxian2 !

@zibaiwan zibaiwan merged commit d3e898f into intel:main Jan 10, 2023
@haoxian2 haoxian2 deleted the coverity-acl-hal-mmd-wrapper branch January 10, 2023 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zibaiwan zibaiwan zibaiwan approved these changes

@pcolberg pcolberg Awaiting requested review from pcolberg

Assignees

@haoxian2 haoxian2

Labels
bug Something isn't working
Projects
None yet
Milestone
2023.2
Development

Successfully merging this pull request may close these issues.
3 participants
@haoxian2 @zibaiwan @pcolberg