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.

Sign up for GitHub

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

Jump to bottom

Security and reliability fix for handling of coverage.data #73

Open

mikel wants to merge 2 commits into markbates:master from reinteractive-open:master

mikel commented Apr 12, 2013

This fixes Issue #43 with a simple rescue block. Code has a covering test.

Also migrates from using eval to ruby Marshal. eval opens a possible attack vector for a sneaky gem.

Thanks for your work on cover_me.

Mikel

Mikel Lindsaar added 2 commits

April 12, 2013 13:45


          Adding fix to return empty data if coverage.data has a syntax error

426a365


          Change from using eval to parse coverage data to Marshal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet