Skip to content

Fix CVE-2022-25168 #26545

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix CVE-2022-25168 #26545

wants to merge 1 commit into from

Conversation

th555555
Copy link

This PR addresses a critical security vulnerability (Command Injection) in the extractTarFileUsingTar method of CompressionUtils.

Modified extractTarFileUsingTar to use a stream-based approach that passes file content through stdin instead of passing file paths to shell commands
This approach eliminates the possibility of command injection via malicious file paths
Maintains the same functionality while improving security

References
https://www.cve.org/CVERecord?id=CVE-2022-25168
apache/hadoop@cae749b

This change is a trivial rework / code cleanup without any test coverage.

…n) in the extractTarFileUsingTar method of CompressionUtils.
@flinkbot
Copy link
Collaborator

flinkbot commented May 10, 2025

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@davidradl
Copy link
Contributor

@th555555 Looking at our process , we need a Jira unless this is a hotfix. I suggest describing in more detail the approach you are taking in resolving this.

@davidradl
Copy link
Contributor

the CI failure appears to be because of running out of space - I guess this is is either due to the fix or infrastructure (disk cleanup). If you think it is due to the disk needing cleating out - I suggest mailing the dev list.

@androidacy-user
Copy link

the CI failure appears to be because of running out of space - I guess this is is either due to the fix or infrastructure (disk cleanup). If you think it is due to the disk needing cleating out - I suggest mailing the dev list.

It's a bot. Don't bother.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants