Better handling of corrupt archives

Author: fkjellberg

src/main/java/org/apache/commons/compress/archivers/lha/LhaArchiveInputStream.java

@@ -166,6 +166,7 @@ Builder setFileSeparatorChar(final char fileSeparatorChar) {
     private static final int HEADER_LEVEL_1_OFFSET_FILENAME = 22;
 
     // Header Level 2
+    private static final int HEADER_LEVEL_2_MINIMUM_HEADER_LENGTH = 26;
     private static final int HEADER_LEVEL_2_OFFSET_HEADER_SIZE = 0;
     private static final int HEADER_LEVEL_2_OFFSET_COMPRESSED_SIZE = 7;
     private static final int HEADER_LEVEL_2_OFFSET_ORIGINAL_SIZE = 11;
@@ -473,7 +474,7 @@ LhaArchiveEntry readHeaderLevel1(ByteBuffer buffer) throws IOException {
      */
     LhaArchiveEntry readHeaderLevel2(ByteBuffer buffer) throws IOException {
         final int headerSize = Short.toUnsignedInt(buffer.getShort(HEADER_LEVEL_2_OFFSET_HEADER_SIZE));
-        if (headerSize < HEADER_GENERIC_MINIMUM_HEADER_LENGTH) {
+        if (headerSize < HEADER_LEVEL_2_MINIMUM_HEADER_LENGTH) {
             throw new ArchiveException("Invalid header level 2 length: %d", headerSize);
         }
 

src/test/java/org/apache/commons/compress/archivers/lha/LhaArchiveInputStreamTest.java

@@ -904,13 +904,13 @@ void testParseHeaderLevel2File() throws IOException {
     void testInvalidHeaderLevel2Length() throws IOException {
         final byte[] data = toByteArray(VALID_HEADER_LEVEL_2_FILE);
 
-        data[0] = 0x10; // Change the first byte to an invalid length
+        data[0] = 25; // Change the first byte to an invalid length
 
         try (LhaArchiveInputStream archive = LhaArchiveInputStream.builder().setInputStream(new ByteArrayInputStream(data)).get()) {
             archive.getNextEntry();
             fail("Expected ArchiveException for invalid header length");
         } catch (ArchiveException e) {
-            assertEquals("Invalid header level 2 length: 16", e.getMessage());
+            assertEquals("Invalid header level 2 length: 25", e.getMessage());
         }
     }