8000 Fix: patch regex vulnerability in filename handling by asiroliu · Pull Request #7887 · infiniflow/ragflow · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix: patch regex vulnerability in filename handling #7887

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 27, 2025
Merged

Fix: patch regex vulnerability in filename handling #7887

merged 1 commit into from
May 27, 2025

Conversation

asiroliu
Copy link
Contributor

What problem does this PR solve?

Regular Expression Injection leading to Denial of Service (ReDoS)

Type of change

  • Bug Fix (non-breaking change which fixes an issue)

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. 🐞 bug Something isn't working, pull request that fix bug. labels May 27, 2025
@asiroliu asiroliu added the ci Continue Integration label May 27, 2025
@asiroliu asiroliu force-pushed the security/GHSA-wqq6-x8g9-f7mh_250527 branch from 8cfad21 to 306e67f Compare May 27, 2025 08:19
@asiroliu asiroliu added ci Continue Integration and removed ci Continue Integration labels May 27, 2025
@asiroliu
Copy link
Contributor Author

Known Issue:
When a filename already contains meaningful parenthesized numbers (e.g., "test(2025).txt"),
the duplicate resolution will incorrectly modify the existing number (e.g., to "test(2026).txt")
rather than following the expected pattern of appending a counter (e.g., "test(2025)(1).txt").

@KevinHuSh KevinHuSh merged commit ff0e829 into infiniflow:main May 27, 2025
1 check passed
@asiroliu asiroliu deleted the security/GHSA-wqq6-x8g9-f7mh_250527 branch May 27, 2025 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
🐞 bug Something isn't working, pull request that fix bug. ci Continue Integration size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@asiroliu @KevinHuSh
0