Commons Compress
  1. Commons Compress
  2. COMPRESS-3

COMPRESS-REDESIGN: Handle JarArchiveInputStream null getNextEntry() gracefully

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Environment:

      N/A

      Description

      When the method JarArchiveInputStream.getNextEntry() is called after the final archive entry was already returned, an Exception is thrown. This is due to the request to the super class ZipArchiveInputStream.getNextEntry() returning null which denotes the archive is at the end. However that null is used to attempt to try to create a new JarArchiveEntry(null) instance. It is here where the exception is thrown. I suggest doing a quick null check then return null if the archive has been fully traversed.

      1. JarArchiveInputStream.patch
        0.6 kB
        Tim Pinet
      2. JarTestCase.patch
        1.0 kB
        Tim Pinet

        Activity

        Tim Pinet created issue -
        Tim Pinet made changes -
        Field Original Value New Value
        Attachment JarArchiveInputStream.patch [ 12391094 ]
        Tim Pinet made changes -
        Attachment JarTestCase.patch [ 12391095 ]
        Torsten Curdt made changes -
        Assignee Torsten Curdt [ tcurdt ]
        Torsten Curdt made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Dennis Lundberg made changes -
        Project Commons Sandbox [ 12310491 ] Commons Compress [ 12310904 ]
        Key SANDBOX-259 COMPRESS-3
        Component/s Compress [ 12311183 ]

          People

          • Assignee:
            Torsten Curdt
            Reporter:
            Tim Pinet
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 10h
              10h
              Remaining:
              Remaining Estimate - 10h
              10h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development