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

Code hardening #2335

Merged
merged 6 commits into from
Feb 18, 2017
Merged

Code hardening #2335

merged 6 commits into from
Feb 18, 2017

Conversation

mgreter
Copy link
Contributor

@mgreter mgreter commented Feb 17, 2017

Addresses +80% of the issues found by coverity.
I don't seem to be able to "restart" the checking though.

@xzyfer
Copy link
Contributor

xzyfer commented Feb 18, 2017

Build frequencies for coverity are restrictive - https://scan.coverity.com/faq#frequency

We discussed in #2325 and in Slack that as a result we mostly use it in addition to my exisiting pre-release sanity check.

I think for now there's a coverity_scan branch that will execute a new build.

@xzyfer xzyfer added this to the 3.5.0.beta.3 milestone Feb 18, 2017
@xzyfer
Copy link
Contributor

xzyfer commented Feb 18, 2017

The CodeKit team is keen to get 3.5 into a release. I'm planning to tag 3.5.0.beta.3 with this and #2334 today to get them over the line and get #2330 into node-sass.

@xzyfer xzyfer mentioned this pull request Feb 18, 2017
@mgreter
Copy link
Contributor Author

mgreter commented Feb 18, 2017

Ran leak-test.pl for this branch and valgrind didn't report anything beside the segfault for issue 2307.

@mgreter mgreter merged commit f6a100e into sass:master Feb 18, 2017
@mgreter mgreter mentioned this pull request Feb 18, 2017
@xzyfer
Copy link
Contributor

xzyfer commented Feb 18, 2017

Nice one!

Please find the latest report on new defect(s) introduced to sass/libsass found with Coverity Scan.

1 new defect(s) introduced to sass/libsass found with Coverity Scan.
25 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan.

New defect(s) Reported-by: Coverity Scan
Showing 1 of 1 defect(s)

@bdkjones
Copy link

Stoked to see static analysis incorporated! It has saved me from shooting myself in my foot many, many times. Especially after refactoring stuff.

@xzyfer
Copy link
Contributor

xzyfer commented Feb 19, 2017

Kicked off another coverity scan. I'll tag 3.5.0.beta.3 pending the results of coverity and my release testing.

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

Successfully merging this pull request may close these issues.

3 participants