Fix backup req bug in h2

87e6224c · zhujiashun · e2556046 · 87e6224c
Commit 87e6224c authored 5 years ago by zhujiashun
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

http2_rpc_protocol.cpp src/brpc/policy/http2_rpc_protocol.cpp +2 -2

No files found.
--- a/src/brpc/policy/http2_rpc_protocol.cpp
+++ b/src/brpc/policy/http2_rpc_protocol.cpp
@@ -1449,10 +1449,10 @@ private:

 void H2UnsentRequest::DestroyStreamUserData(SocketUniquePtr& sending_sock,
                                            Controller* cntl,
-                                            int /*error_code*/,
+                                            int error_code,
                                            bool /*end_of_rpc*/) {
    RemoveRefOnQuit deref_self(this);
-    if (sending_sock != NULL && cntl->ErrorCode() != 0) {
+    if (sending_sock != NULL && (cntl->ErrorCode() != 0 || error_code != 0)) {
        CHECK_EQ(cntl, _cntl);
        std::unique_lock<butil::Mutex> mu(_mutex);
        _cntl = NULL;