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

Deadlock in Test/ClientUpdatesParamsOnGoAway on failure #4537

Closed
dustin-ward opened this issue Jun 11, 2021 · 0 comments · Fixed by #4534
Closed

Deadlock in Test/ClientUpdatesParamsOnGoAway on failure #4537

dustin-ward opened this issue Jun 11, 2021 · 0 comments · Fixed by #4534

Comments

@dustin-ward
Copy link
Contributor

What version of gRPC are you using?

1.38.0

What version of Go are you using (go version)?

1.16.4

What operating system (Linux, Windows, …) and version?

Linux (Ubuntu 20.04.2 LTS) & zOS

What did you do?

An unrelated network issue caused this test to enter the second conditional statement. Forcing the program to enter this conditional will result in the same issue.

What did you expect to see?

The test should unlock the associated mutex before returning a fatal error message, so the program can properly end execution and report the actual errors.

What did you see instead?

The mutex is never unlocked, so the program will hang at this point.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant