Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-28 Thread via GitHub
yakovsh commented on PR #442: URL: https://github.com/apache/commons-compress/pull/442#issuecomment-1830050151 Thank you! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-28 Thread via GitHub
garydgregory merged PR #442: URL: https://github.com/apache/commons-compress/pull/442 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-27 Thread via GitHub
yakovsh commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1406841978 ## src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveConstants.java: ## @@ -26,7 +26,7 @@ public final class DumpArchiveConstants { *

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-23 Thread via GitHub
garydgregory commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1402651501 ## src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveConstants.java: ## @@ -26,7 +26,7 @@ public final class DumpArchiveConstants {

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-22 Thread via GitHub
yakovsh commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1402701318 ## src/main/java/org/apache/commons/compress/archivers/dump/TapeInputStream.java: ## @@ -294,6 +294,9 @@ public void resetBlockSize(final int recsPerBlock,

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-22 Thread via GitHub
garydgregory commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1402651501 ## src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveConstants.java: ## @@ -26,7 +26,7 @@ public final class DumpArchiveConstants {

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-22 Thread via GitHub
garydgregory commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1402647680 ## src/main/java/org/apache/commons/compress/archivers/dump/TapeInputStream.java: ## @@ -294,6 +294,9 @@ public void resetBlockSize(final int recsPerBlock,

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-21 Thread via GitHub
yakovsh commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1401261090 ## src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java: ## @@ -526,7 +526,7 @@ private void readDirectoryEntry(DumpArchiveEntry

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-21 Thread via GitHub
yakovsh commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1401261090 ## src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java: ## @@ -526,7 +526,7 @@ private void readDirectoryEntry(DumpArchiveEntry

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-21 Thread via GitHub
garydgregory commented on code in PR #442: URL: https://github.com/apache/commons-compress/pull/442#discussion_r1401169457 ## src/main/java/org/apache/commons/compress/archivers/dump/DumpArchiveInputStream.java: ## @@ -526,7 +526,7 @@ private void

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-15 Thread via GitHub
yakovsh commented on PR #442: URL: https://github.com/apache/commons-compress/pull/442#issuecomment-1813330529 @garydgregory I re-based from master and fixed issues surfaced by checkstyle -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-15 Thread via GitHub
yakovsh commented on PR #442: URL: https://github.com/apache/commons-compress/pull/442#issuecomment-1813079201 Sure, thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-15 Thread via GitHub
garydgregory commented on PR #442: URL: https://github.com/apache/commons-compress/pull/442#issuecomment-1813047068 Please hold off creating PRs while I add checkstyle to the build, it will just cause conflicts; -) -- This is an automated message from the Apache Git Service. To respond

[PR] COMPRESS-632: Fixes for dump file parsing [commons-compress]

2023-11-15 Thread via GitHub
yakovsh opened a new pull request, #442: URL: https://github.com/apache/commons-compress/pull/442 This PR adds a few fixes and unit tests for dump file parsing, related to fuzzing work in COMPRESS-632. -- This is an automated message from the Apache Git Service. To respond to the