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

Ensure from < to when tracing chain #17460

Merged
merged 1 commit into from Aug 21, 2018
Merged

Conversation

holiman
Copy link
Contributor

@holiman holiman commented Aug 21, 2018

A bug received via bug bounty, credits to Chen Nan.

@holiman holiman requested a review from karalabe as a code owner August 21, 2018 07:50
@Thor-Heimdallr
Copy link

Thor-Heimdallr commented Aug 21, 2018 via email

@holiman
Copy link
Contributor Author

holiman commented Aug 21, 2018

You're welcome to apply for a CVE number for this, as I wrote to you we do not consider it a vulnerability. The debug namespace is for debugging, and should not be exposed to untrusted parties.

Copy link
Member

@karalabe karalabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@karalabe karalabe added this to the 1.8.14 milestone Aug 21, 2018
@karalabe karalabe merged commit ecca49e into ethereum:master Aug 21, 2018
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.

None yet

3 participants