4 ms·
Examples of the actual errors found would be far more interesting.
by lelf 12y ago
Examples of the actual errors found would be far more interesting.
- sounds 12y agoI figured the last sentence of the post, specifically "there are a few interesting cases," would mean he will post some specific examples next time. Crossing my fingers.
- pgeorgi 12y agoWe recently started using Coverity Scan for coreboot. So far I mostly took on the low hanging fruits, relevant commits can be found through http://review.coreboot.org/#/q/message:%22Coverity+Scan%22 http://review.coreboot.org/#/q/message:%22Coverity+Scan%22 We'll soon start using clang's scan-build systematically, too.
- justinmk 12y agoIf it's anything like the Neovim coverity report, it's probably not particularly interesting. But it is extremely helpful, and it's impressive what static analysis can find in a legacy C codebase. Here's the Neovim clang analysis: http://neovim.org/doc/build-reports/clang/ http://neovim.org/doc/build-reports/clang/ Click on any of the "View Report" links to see a line-by-line step-through of each suspected bug. (Coverity doesn't allow public access to reports AFAIK, so can't link to it.)
- seren 12y agoSorry to highjack the thread, but if I want to fix some neovim Coverity issues, where should I register / who should I ask ? I have already had a look to your waffle but it was not crystal clear. Just creating a Coverity account will allow me to access the report ? Thanks in advance.
- davis 12y agoLogin then go to https://scan.coverity.com/projects/2227 https://scan.coverity.com/projects/2227 and request access. If you want to fix Coverity issues, request access at the Contributor level. For more info on Coverity fixes and conventions, check out the Wiki page: https://github.com/neovim/neovim/wiki/Contributing#coverity https://github.com/neovim/neovim/wiki/Contributing#coverity
- binarymax 12y agoClicking into some other posts in the same topic, there are a couple examples: if (condition) bar(); else bar(); And switch (foo) { case 0: bar(); ;; case 1: misc(); break; }