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

Fix a regex denial of service in trim() and rtrim() #556

Merged

chriso merged 2 commits into master from chriso/trim-redos

Jul 21, 2016

Collaborator

chriso commented Jul 21, 2016

Reported by @dpkirchner.

chriso added 2 commits

July 21, 2016 12:47


          Rewrite trim() to use rtrim() + ltrim()

22426b0


          Rewrite rtrim() to avoid a regex DoS

057d3b0

coveralls commented Jul 21, 2016 •

edited

Loading

Coverage increased (+0.002%) to 99.093% when pulling 057d3b0 on chriso/trim-redos into 47a1573 on master.

chriso merged commit 351fa77 into master

chriso deleted the chriso/trim-redos branch

July 21, 2016 03:05

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

Labels

None yet