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

Coverity Static Code Analysis Resutls #333

Closed
GoogleCodeExporter opened this issue Aug 18, 2015 · 4 comments
Closed

Coverity Static Code Analysis Resutls #333

GoogleCodeExporter opened this issue Aug 18, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

The attached document are high sightings found by a Coverity scan ran today for 
Vim 7.4.  Some of these may be false positives, but it may be good for others 
to take a look.

Original issue reported on code.google.com by john.l.w...@intel.com on 19 Feb 2015 at 10:58

@GoogleCodeExporter
Copy link
Author

Thanks! What patch level is this? Or is it 7.4.0? Is that all of them or did 
you share just a selection?

Note, there is a cov-format-errors command that can generate an error list 
usable by vim's quickfix. I wrote a compiler plugin that sets an appropriate 
errorformat: http://www.vim.org/scripts/script.php?script_id=3936

For some items you've provided only the screenshot of the final problem, but 
not the logic needed to reach that problem spot, which I've found to be a 
useful part of the Coverity results. That same cov-format-errors command can 
provide an HTML dump that mirrors what you'd see in the Coverity server. I 
think the info in this is equivalent to that in the error list but I'm not 
certain on that.

Perhaps the error list or html dump would be useful to address some of these.

Original comment by fritzoph...@gmail.com on 20 Feb 2015 at 2:41

@GoogleCodeExporter
Copy link
Author

Although I appreciate reporting possible issues, it's a lot easier to deal with 
them by creating a separate issue for every problem.  Please split it up, so 
that we can mark each issue with specific comments and close the ones that get 
fixed.

Original comment by brammool...@gmail.com on 20 Feb 2015 at 6:48

@GoogleCodeExporter
Copy link
Author

Ok, sounds good.  I recommend closing this one then and I can go back and 
dispatch each applicable issue in a better format per fritzophrenic's comments. 
 Thanks!

Original comment by john.l.w...@intel.com on 20 Feb 2015 at 11:00

@GoogleCodeExporter
Copy link
Author

Okay, closing as invalid then. Feel free to open new issues for individual 
issues. This allows for easier tracking the problems.

Original comment by chrisbr...@googlemail.com on 25 Feb 2015 at 9:32

  • Changed state: Invalid

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