Uploaded image for project: 'CXF'
  1. CXF
  2. CXF-2816

CachedOutputStream leaves behind temp files if resetOut() is called

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.2.8
    • 2.2.9
    • Core
    • None
    • Windows

    • Novice

    Description

      CachedOutputStream's resetOut() has a section of code where it is possible to open a FileInputStream without closing it. This happens if the stream is not in memory, and copyOldContent is false.

      I noticed this when I added a "LoggingOutInterceptor". When large messages were logged, temp files would accumulate in the temp directory. Applying the attached patch solved the problem.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            gmazza Glen Mazza
            piepera aaron pieper
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment