GitHub user marshalium opened a pull request:

    https://github.com/apache/thrift/pull/512

    Close transport after reading an invalid frame size

    This is necessary because after reading the invalid
    frame size from the underlying transport the transport
    is left in a bad state. Any following reads will
    likely throw more invalid frame size exceptions and
    if they don’t they will be reading corrupted messages.
    
    Closing forces the caller to discard the connection and
    get a new one.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/marshalium/thrift 
marshalium/close_on_frame_size_error

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/thrift/pull/512.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #512
    
----
commit a02c7e3a91a2f75f2de3dd9cb4d0ed25a1582369
Author: Marshall Scorcio <mscor...@swiftype.com>
Date:   2015-06-05T22:03:30Z

    Close the transport after reading an invalid frame size
    
    This is necessary because after reading the invalid
    frame size from the underlying transport the transport
    is left in a bad state. Any following reads will
    likely throw more invalid frame size exceptions and
    if they don’t they will be reading corrupted messages.
    
    Closing forces the caller to discard the connection and
    get a new one.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to