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

fix the static scan issues #311

Merged
merged 1 commit into from
Mar 6, 2023
Merged

fix the static scan issues #311

merged 1 commit into from
Mar 6, 2023

Conversation

xianju6x
Copy link
Contributor

@xianju6x xianju6x commented Feb 15, 2023 •
edited by YadongQi
Loading

Signed-off-by: xianju6x xianjunx.zheng@intel.com

YadongQi
YadongQi reviewed Feb 15, 2023
View reviewed changes
src/guest/vm_flash.cc Outdated Show resolved Hide resolved
src/guest/vm_flash.cc Outdated Show resolved Hide resolved
src/utils/utils.cc Outdated Show resolved Hide resolved
src/utils/utils.cc Outdated Show resolved Hide resolved
@xianju6x xianju6x force-pushed the master branch 2 times, most recently from b7be35b to ef3febb Compare February 17, 2023 02:38
@YadongQi YadongQi changed the title fix the issues scanned by Coverity fix the static scan issues Feb 27, 2023
@xianju6x xianju6x force-pushed the master branch 3 times, most recently from 4674291 to 763c7d4 Compare February 27, 2023 06:02
YadongQi
YadongQi requested changes Feb 27, 2023
View reviewed changes
src/guest/vm_process.cc Outdated Show resolved Hide resolved
src/guest/vm_process.cc Outdated Show resolved Hide resolved
src/services/client.cc Outdated Show resolved Hide resolved
src/utils/utils.cc Outdated Show resolved Hide resolved
@xianju6x xianju6x force-pushed the master branch 2 times, most recently from b0bdc34 to 23632e0 Compare February 27, 2023 06:53
YadongQi
YadongQi reviewed Feb 27, 2023
View reviewed changes
src/utils/utils.cc Outdated Show resolved Hide resolved
YadongQi
YadongQi reviewed Feb 27, 2023
View reviewed changes
src/utils/utils.cc Outdated Show resolved Hide resolved
YadongQi
YadongQi reviewed Feb 28, 2023
View reviewed changes
src/guest/aaf.cc Outdated Show resolved Hide resolved
src/guest/tui.cc Outdated Show resolved Hide resolved
src/guest/tui.cc Outdated Show resolved Hide resolved
YadongQi
YadongQi previously approved these changes Mar 6, 2023
View reviewed changes
@xianju6x @YadongQi
fix the issues scanned by Coverity
53cb563 
Signed-off-by: xianju6x <xianjunx.zheng@intel.com>
@sysopenci
Copy link

Building VM-Manager Code. For more details refer below link:
job/satg-aee-android-ci/job/VM-MANAGER-PR-FLOW/129/

@sysopenci
Copy link

Building VM-Manager Code. For more details refer below link:
job/satg-aee-android-ci/job/VM-MANAGER-PR-FLOW/130/

@sysopenci
Copy link

Build successful.
job/satg-aee-android-ci/job/VM-MANAGER-PR-FLOW/129/

@sysopenci
Copy link

Please find the Deb Package Path :
artifactory/dev-automation-ba-local/vm-manager-binaries/311/PR/53cb563

@sysopenci
Copy link

Build successful.
job/satg-aee-android-ci/job/VM-MANAGER-PR-FLOW/130/

@sysopenci
Copy link

Please find the Deb Package Path :
artifactory/dev-automation-ba-local/vm-manager-binaries/311/PR/53cb563

@sysopenci
Copy link

Building VM-Manager Code. For more details refer below link:
job/satg-aee-android-ci/job/VM-MANAGER-PR-MERGE-FLOW/88/

@sysopenci
Copy link

Build successful.
job/satg-aee-android-ci/job/VM-MANAGER-PR-MERGE-FLOW/88/

@sysopenci
Copy link

Please find the Deb Package Path :
artifactory/dev-automation-ba-local/vm-manager-binaries/311/MR/53cb563

@sysopenci sysopenci merged commit fc1a22e into projectceladon:master Mar 6, 2023
@sysopenci sysopenci mentioned this pull request Mar 6, 2023
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YadongQi YadongQi YadongQi approved these changes

Assignees
No one assigned
Labels
Test Passed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@xianju6x @sysopenci @YadongQi