Deadlock fix for GRPCCall

src/objective-c/GRPCClient/GRPCCall.m

@@ -539,26 +539,24 @@ const char *kCFStreamVarName = "grpc_cfstream";
 
 #pragma mark Finish
 
+// This function should support being called within a @synchronized(self) block in another function
+// Should not manipulate _requestWriter for deadlock prevention.
 - (void)finishWithError:(NSError *)errorOrNil {
-  GRXConcurrentWriteable *copiedResponseWriteable = nil;
-
   @synchronized(self) {
     if (_state == GRXWriterStateFinished) {
       return;
     }
     _state = GRXWriterStateFinished;
-    copiedResponseWriteable = _responseWriteable;
-
-    // If the call isn't retained anywhere else, it can be deallocated now.
-    _retainSelf = nil;
-  }
 
     if (errorOrNil) {
-    [copiedResponseWriteable cancelWithError:errorOrNil];
+      [_responseWriteable cancelWithError:errorOrNil];
     } else {
-    [copiedResponseWriteable enqueueSuccessfulCompletion];
+      [_responseWriteable enqueueSuccessfulCompletion];
+    }
+
+    // If the call isn't retained anywhere else, it can be deallocated now.
+    _retainSelf = nil;
   }
-  _requestWriter.state = GRXWriterStateFinished;
 }
 
 - (void)cancel {
@@ -572,19 +570,7 @@ const char *kCFStreamVarName = "grpc_cfstream";
                                      userInfo:@{NSLocalizedDescriptionKey : @"Canceled by app"}]];
     [_wrappedCall cancel];
   }
-}
+  _requestWriter.state = GRXWriterStateFinished;
-
-- (void)maybeFinishWithError:(NSError *)errorOrNil {
-  BOOL toFinish = NO;
-  @synchronized(self) {
-    if (_finished == NO) {
-      _finished = YES;
-      toFinish = YES;
-    }
-  }
-  if (toFinish == YES) {
-    [self finishWithError:errorOrNil];
-  }
 }
 
 - (void)dealloc {
@@ -647,6 +633,7 @@ const char *kCFStreamVarName = "grpc_cfstream";
                                               }]];
           [strongSelf->_wrappedCall cancel];
         }
+        strongSelf->_requestWriter.state = GRXWriterStateFinished;
       } else {
         @synchronized(strongSelf) {
           [strongSelf->_responseWriteable enqueueValue:data
@@ -818,6 +805,7 @@ const char *kCFStreamVarName = "grpc_cfstream";
             error = [NSError errorWithDomain:error.domain code:error.code userInfo:userInfo];
           }
           [strongSelf finishWithError:error];
+          strongSelf->_requestWriter.state = GRXWriterStateFinished;
         }
       }];
 }
@@ -841,7 +829,7 @@ const char *kCFStreamVarName = "grpc_cfstream";
                                     callOptions:_callOptions];
 
     if (_wrappedCall == nil) {
-      [self maybeFinishWithError:[NSError errorWithDomain:kGRPCErrorDomain
+      [self finishWithError:[NSError errorWithDomain:kGRPCErrorDomain
                                                 code:GRPCErrorCodeUnavailable
                                             userInfo:@{
                                                        NSLocalizedDescriptionKey :
@@ -971,6 +959,7 @@ const char *kCFStreamVarName = "grpc_cfstream";
                                             NSLocalizedDescriptionKey : @"Connectivity lost."
                                           }]];
     }
+    strongSelf->_requestWriter.state = GRXWriterStateFinished;
   }
 }