Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
0.10.0
-
None
-
Patch Available
-
Patch, Important
Description
The TSSLSocket class did not handle large messages, because a underlying TCP socket may signal bytes received, while SSL_read() may not have bytes available. After maxretries (5) the function returned -1, which got interpreted as unsigned integer for read bytes.
Futher the waitForEvent methode, did only set THRIFT_POLLIN or THRIFT_POLLOUT, but it gets used where SSL needs to send AND receive bytes for some operations (like close). So in the case of write wanted, THRIFT_POLLIN is also set to cover these read/write operations.
Pullrequest for master and 0.10.0 branch will follow.
Attachments
Attachments
Issue Links
- is duplicated by
-
THRIFT-4366 upgrade thrift lib to 0.10 ,coredump at readMessageBegin
- Closed
- links to
patches for master and 0.10.0