grpc: prevent deadlock in Test/ClientUpdatesParamsAfterGoAway on fail…

…ure (#4534)
grpc · Jun 11, 2021 · 2d3b1f9 · 2d3b1f9
1 parent 6351a55
commit 2d3b1f9
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/clientconn_test.go b/clientconn_test.go
@@ -735,16 +735,15 @@ func (s) TestClientUpdatesParamsAfterGoAway(t *testing.T) {
 		time.Sleep(10 * time.Millisecond)
 		cc.mu.RLock()
 		v := cc.mkp.Time
+		cc.mu.RUnlock()
 		if v == 20*time.Second {
 			// Success
-			cc.mu.RUnlock()
 			return
 		}
 		if ctx.Err() != nil {
 			// Timeout
 			t.Fatalf("cc.dopts.copts.Keepalive.Time = %v , want 20s", v)
 		}
-		cc.mu.RUnlock()
 	}
 }