search

gerddie / ginkgocadx

Advanced DICOM viewer
131 stars 38 forks source link

Coverity has discovered 154 problems in the code #15

Closed gerddie closed 8 years ago

gerddie commented 8 years ago

This bug is here to reference changes to these in the commits.

onlyjob commented 8 years ago

What's "Coverity"?

onlyjob commented 8 years ago

OK, this seems relevant: https://scan.coverity.com/github

gerddie commented 8 years ago

The overview of the current build is here.

BTW: Github integration is not very relevant at this point, because this is mostly for using travis-ci to build and upload, and the travis-ci build environment uses a rather old Ubuntu (14.04 at best), which has an old version of ITK.

onlyjob commented 8 years ago

The overview of the current build is here.

That page requires sign-in unfortunately... Also Coverity is proprietary... :( Does it have free alternatives?

gerddie commented 8 years ago

Here is the the public page. Well, Coverity is free-as-in-beer for developers of free software ... but yes, one has to run their proprietary tool.

The only free alternative I can think of is using clang to compile the code with the static analyzer, it's not as thorough though.

gerddie commented 8 years ago

Well, and some errors would also be caught by valgrind.

onlyjob commented 8 years ago

Thought I'd mention cppcheck. :)

gerddie commented 8 years ago

All issues resolved with the exception of #16.