Uploaded image for project: 'ShrinkWrap'
  1. ShrinkWrap
  2. SHRINKWRAP-470

SeekableInMemoryByteChannel.truncate misimplements the contract

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Major
    • 1.2.1
    • None
    • None
    • None

    Description

      At the tail end of truncate, is this logic:

      // If we've been given a size greater than we are
      if (newSize > currentSize) {
          // Reset the position only
          this.position = newSize;
      }
      

      which is not what the API says: "If the given size is less than the current size then the entity is truncated, discarding any bytes beyond the new end. If the given size is greater than or equal to the current size then the entity is not modified."

      I think you wanted this instead, which should be inside the above array truncation:

      // If we've been given a size less than we are
      if (newSize < currentSize) {
          // Reset the size only
          this.size = newSize;
      }
      

      Attachments

        Activity

          People

            mmatloka Michal Matloka (Inactive)
            undertheflowerpot Under Flowerpot (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: