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

Don't give out "Worth it!" if it wasn't worth it #73

Merged
merged 2 commits into from
Jun 19, 2016

Conversation

TomyLobo
Copy link
Collaborator

The first commit fixes an issue that is definitely exploitable and has been exploited.
The second commit reverses a deliberate change introduced by 3d3ff3f

I think we should add the first commit. What's your opinion on the second?

@TomyLobo TomyLobo added this to the v1.2.2 milestone Jun 19, 2016
@TomyLobo TomyLobo changed the title Don't give out "worth it" so easily Don't give out "Worth it!" so easily Jun 19, 2016
@BSVino
Copy link
Owner

BSVino commented Jun 19, 2016

Ship

@TomyLobo TomyLobo force-pushed the no-non-superfall-damage-worthit branch from 1c6a34a to 2a4c336 Compare June 19, 2016 18:27
@TomyLobo TomyLobo force-pushed the no-non-superfall-damage-worthit branch from 2a4c336 to 76de3e3 Compare June 19, 2016 18:35
@TomyLobo TomyLobo changed the title Don't give out "Worth it!" so easily Don't give out "Worth it!" if it wasn't worth it Jun 19, 2016
@TomyLobo TomyLobo merged commit 38ceae2 into develop Jun 19, 2016
@TomyLobo TomyLobo deleted the no-non-superfall-damage-worthit branch June 19, 2016 18:41
@TomyLobo TomyLobo restored the no-non-superfall-damage-worthit branch June 19, 2016 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants