Skip to content

Acquire mutex before modifying DNSHandler #7316

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
Closed
masaori335 wants to merge 1 commit into from
Closed

Acquire mutex before modifying DNSHandler #7316

masaori335 wants to merge 1 commit into from

Conversation

@masaori335
Copy link
Contributor

@masaori335 masaori335 commented Nov 11, 2020

Fix #7315.

@masaori335 masaori335 added the DNS label Nov 11, 2020
@masaori335 masaori335 added this to the 10.0.0 milestone Nov 11, 2020
@masaori335 masaori335 self-assigned this Nov 11, 2020
@masaori335 masaori335 marked this pull request as draft November 11, 2020 05:54
@randall
Copy link
Contributor

randall commented Nov 11, 2020

[approve ci autest]

@masaori335 masaori335 mentioned this pull request Nov 13, 2020
Merged
@masaori335
Copy link
Contributor Author

masaori335 commented Nov 16, 2020

#7321 looks better.

@masaori335 masaori335 closed this Nov 16, 2020
@zwoop zwoop removed this from the 10.0.0 milestone Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@masaori335 masaori335

Labels

DNS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8.1.0 Crash on write_dns

3 participants

@masaori335 @randall @zwoop