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.

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

Regex hardening to fix CVE-2018-12356. #1147

Merged
merged 2 commits into from Jun 20, 2018
Merged

Regex hardening to fix CVE-2018-12356. #1147

merged 2 commits into from Jun 20, 2018

Conversation

PenTesting
Copy link
Contributor

No description provided.

@sickpig
Copy link
Collaborator

sickpig commented Jun 18, 2018

We not currently using the script this PR is fixing, but we could start using in the the future so I guess is a good practice to have it fixed

@jasonbcox jasonbcox mentioned this pull request Jun 18, 2018
Closed
@dagurval
Copy link
Collaborator

bitcoin/bitcoin#13479

@gandrewstone gandrewstone merged commit 056f4f1 into BitcoinUnlimited:dev Jun 20, 2018
@PenTesting PenTesting deleted the patch-1 branch June 20, 2018 16:17
PenTesting added a commit to PenTesting/bitcoin-abc that referenced this pull request Jun 1, 2019
@PenTesting
Fix CVE-2018-12356 by hardening the regex
28fa7e2 
Reference:
bitcoin/bitcoin#13479
BitcoinUnlimited/BitcoinUnlimited#1147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sickpig sickpig sickpig approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@PenTesting @sickpig @dagurval @gandrewstone