[thrift] branch master updated: THRIFT-5725: Fix the comparison check for bio_get_fd Client: cpp Patch: Prasenjit Sarkar

[jensg]

This is an automated email from the ASF dual-hosted git repository.

jensg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/thrift.git


The following commit(s) were added to refs/heads/master by this push:
     new bccca9301 THRIFT-5725: Fix the comparison check for bio_get_fd Client: 
cpp Patch: Prasenjit Sarkar
bccca9301 is described below

commit bccca9301dedd6403062ef6be4c56533f889e5b5
Author: Prasenjit Sarkar <prasenjit.sar...@datos.io>
AuthorDate: Thu Jul 6 21:26:32 2023 -0700

    THRIFT-5725: Fix the comparison check for bio_get_fd
    Client: cpp
    Patch: Prasenjit Sarkar
    
    This closes #2831
---
 lib/cpp/src/thrift/transport/TSSLSocket.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/cpp/src/thrift/transport/TSSLSocket.cpp 
b/lib/cpp/src/thrift/transport/TSSLSocket.cpp
index eb96a73a6..26ae2beda 100644
--- a/lib/cpp/src/thrift/transport/TSSLSocket.cpp
+++ b/lib/cpp/src/thrift/transport/TSSLSocket.cpp
@@ -825,7 +825,7 @@ unsigned int TSSLSocket::waitForEvent(bool wantRead) {
     throw TSSLException("SSL_get_?bio returned nullptr");
   }
 
-  if (BIO_get_fd(bio, &fdSocket) <= 0) {
+  if (BIO_get_fd(bio, &fdSocket) < 0) {
     throw TSSLException("BIO_get_fd failed");
   }