Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8, 0.9
    • Fix Version/s: 0.9
    • Component/s: Delphi - Library
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      There are a few issues with the TFramedTransport implementation.

      • The offset "off" is ignored, instead always 0 is used fpor reads and writes
      • Trying to write an empty byte array results in range check exceptions

        Activity

        Hide
        Jake Farrell added a comment -

        thanks for the patch, committed

        Show
        Jake Farrell added a comment - thanks for the patch, committed
        Hide
        Hudson added a comment -

        Integrated in Thrift #425 (See https://builds.apache.org/job/Thrift/425/)
        THRIFT-1537:TFramedTransport issues
        Client: delphi
        Patch: Jens Geyer

        TFramedTransport fixes for:

        • The offset "off" is ignored, instead always 0 is used fpor reads and writes
        • Trying to write an empty byte array results in range check exceptions (Revision 1303637)

        Result = SUCCESS
        jfarrell : http://svn.apache.org/viewvc/?view=rev&rev=1303637
        Files :

        • /thrift/trunk/lib/delphi/src/Thrift.Protocol.pas
        • /thrift/trunk/lib/delphi/src/Thrift.Transport.pas
        Show
        Hudson added a comment - Integrated in Thrift #425 (See https://builds.apache.org/job/Thrift/425/ ) THRIFT-1537 :TFramedTransport issues Client: delphi Patch: Jens Geyer TFramedTransport fixes for: The offset "off" is ignored, instead always 0 is used fpor reads and writes Trying to write an empty byte array results in range check exceptions (Revision 1303637) Result = SUCCESS jfarrell : http://svn.apache.org/viewvc/?view=rev&rev=1303637 Files : /thrift/trunk/lib/delphi/src/Thrift.Protocol.pas /thrift/trunk/lib/delphi/src/Thrift.Transport.pas

          People

          • Assignee:
            Jens Geyer
            Reporter:
            Jens Geyer
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development