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

Generate Coverity static code check report on edk2-test and fix (Bugzilla Bug 4844) #233

Open
tianocore-issues opened this issue Sep 6, 2024 · 8 comments

Comments

@tianocore-issues
Copy link

This issue was created automatically with bugzilla2github

Bugzilla Bug 4844

Date: 2024-09-06T16:35:09+00:00
From: Edhay <<edhaya.chandran>>
To: unassigned <>
CC: vincent.stehle

Last updated: 2024-11-07T10:45:52+00:00

@tianocore-issues
Copy link
Author

Comment 23388

Date: 2024-09-06 16:35:09 +0000
From: Edhay <<edhaya.chandran>>

  • Industry Specification: ---
  • Release Observed: EDK II Master
  • Releases to Fix: EDK II Master
  • Target OS: ---
  • Bugzilla Assignee(s): unassigned <>

Generate the Coverity static code check report on edk2-test code and fix high impact and high return errors/warnings.

@tianocore-issues
Copy link
Author

Comment 23424

Date: 2024-10-03 10:36:11 +0000
From: Edhay <<edhaya.chandran>>

Vincent to provide a Coverity run report on the latest edk2-test code.

To plan: Once the issues have been resolved, the UEFI-SCT must be tested for any impact due to the fixes.

@tianocore-issues
Copy link
Author

Comment 23428

Date: 2024-10-04 09:23:58 +0000
From: Vincent Stehlé <<vincent.stehle>>

Created attachment 1542
Coverity analysis summary

Attachment: BZ-1542-summary.txt

@tianocore-issues
Copy link
Author

Comment 23429

Date: 2024-10-04 09:24:56 +0000
From: Vincent Stehlé <<vincent.stehle>>

Created attachment 1543
Coverity analysis results in oneline text format

Attachment: BZ-1543-oneline.zip

@tianocore-issues
Copy link
Author

Comment 23430

Date: 2024-10-04 09:25:52 +0000
From: Vincent Stehlé <<vincent.stehle>>

Created attachment 1544
Coverity analysis results in JSON format

Attachment: BZ-1544-json.zip

@tianocore-issues
Copy link
Author

Comment 23431

Date: 2024-10-04 09:26:55 +0000
From: Vincent Stehlé <<vincent.stehle>>

Created attachment 1545
Coverity analysis results in MSVS format

Attachment: BZ-1545-msvs.zip

@tianocore-issues
Copy link
Author

Comment 23432

Date: 2024-10-04 09:32:14 +0000
From: Vincent Stehlé <<vincent.stehle>>

Hi,

I have built the SCT for X86 and AARCH64 under Coverity. The results in various formats are attached to this issue.

Due to the attachment size limit of 1 MB, it is sadly not practicable to attach the results in HTML format here, as they weight ~80 MB.

This is with:

  • Coverity 2023.9.0
  • edk2 91d806917fd2
  • edk2-test 16c552d

@tianocore-issues
Copy link
Author

Comment 23519

Date: 2024-11-07 10:45:52 +0000
From: Edhay <<edhaya.chandran>>

Thank you, Vincent for the reports.

Ann to take up the initial analysis of the reports.
The fix effort shall be shared among community members.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant