Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31805
Change subject: Docs/project_ideas: Add coverity scan cleanup project ......................................................................
Docs/project_ideas: Add coverity scan cleanup project
Change-Id: I16d9a7f7088254c5c207adc9299a8525bf38199f Signed-off-by: Patrick Georgi pgeorgi@google.com --- M Documentation/contributing/project_ideas.md 1 file changed, 20 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/31805/1
diff --git a/Documentation/contributing/project_ideas.md b/Documentation/contributing/project_ideas.md index e7dd3a7..117dcce 100644 --- a/Documentation/contributing/project_ideas.md +++ b/Documentation/contributing/project_ideas.md @@ -152,3 +152,23 @@
### Mentors * Patrick Georgi patrick@georgi.software + +## Make coreboot coverity clean +coreboot and several other of our projects are automatically tested +using Synopsys' free "Coverity Scan" service. While some fare pretty +good, like [em100](https://scan.coverity.com/projects/em100) at 0 known +defects, there are still many open issues in other projects, most notably +[coreboot](https://scan.coverity.com/projects/coreboot) itself (which +is also the largest codebase). + +Not all of the reports are actual issues, but the project benefits a +lot if the list of unhandled reports is down to 0 because that provides +a baseline when future changes reintroduce new issues: it's easier to +triage and handle a list of 5 issues rather than more than 350. + +This project would be going through all reports and handling them +appropriately: Figure out if reports are valid or not and mark them +as such. For valid reports, provide patches to fix the underlying issue. + +### Mentors +* Patrick Georgi patrick@georgi.software
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31805 )
Change subject: Docs/project_ideas: Add coverity scan cleanup project ......................................................................
Patch Set 1: Code-Review+2
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31805 )
Change subject: Docs/project_ideas: Add coverity scan cleanup project ......................................................................
Patch Set 1: Code-Review+2
Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31805 )
Change subject: Docs/project_ideas: Add coverity scan cleanup project ......................................................................
Docs/project_ideas: Add coverity scan cleanup project
Change-Id: I16d9a7f7088254c5c207adc9299a8525bf38199f Signed-off-by: Patrick Georgi pgeorgi@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/31805 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Rudolph siro@das-labor.org Reviewed-by: Angel Pons th3fanbus@gmail.com --- M Documentation/contributing/project_ideas.md 1 file changed, 20 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Patrick Rudolph: Looks good to me, approved Angel Pons: Looks good to me, approved
diff --git a/Documentation/contributing/project_ideas.md b/Documentation/contributing/project_ideas.md index 9a2d414..da51246 100644 --- a/Documentation/contributing/project_ideas.md +++ b/Documentation/contributing/project_ideas.md @@ -146,3 +146,23 @@
### Mentors * Patrick Georgi patrick@georgi.software + +## Make coreboot coverity clean +coreboot and several other of our projects are automatically tested +using Synopsys' free "Coverity Scan" service. While some fare pretty +good, like [em100](https://scan.coverity.com/projects/em100) at 0 known +defects, there are still many open issues in other projects, most notably +[coreboot](https://scan.coverity.com/projects/coreboot) itself (which +is also the largest codebase). + +Not all of the reports are actual issues, but the project benefits a +lot if the list of unhandled reports is down to 0 because that provides +a baseline when future changes reintroduce new issues: it's easier to +triage and handle a list of 5 issues rather than more than 350. + +This project would be going through all reports and handling them +appropriately: Figure out if reports are valid or not and mark them +as such. For valid reports, provide patches to fix the underlying issue. + +### Mentors +* Patrick Georgi patrick@georgi.software
-
Angel Pons (Code Review) -
Patrick Georgi (Code Review)
-
Patrick Rudolph (Code Review)