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

Reflected_XSS_All_Clients @ /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java #132

Open
cyates-checkmarx opened this issue Jun 23, 2023 · 0 comments

Comments

@cyates-checkmarx
Copy link
Owner

cyates-checkmarx commented Jun 23, 2023

Checkmarx (SAST): Reflected_XSS_All_Clients
Security Issue: Read More about Reflected_XSS_All_Clients
Checkmarx Project: cyates-checkmarx/JVL
Repository URL: https://github.com/cyates-checkmarx/JVL
Branch: master
Scan ID: 6bf2c567-d73e-490c-ac86-a35dd14d497b


The method processRequest embeds untrusted data in generated output with print, at line 54 of /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java. This untrusted data is embedded into the output without proper sanitization or encoding, enabling an attacker to inject malicious code into the generated web-page.

The attacker would be able to alter the returned web page by simply providing modified data in the user input getInputStream, which is read by the processRequest method at line 44 of /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java. This input then flows through the code straight to the output web page, without sanitization. 

This can enable a Reflected Cross-Site Scripting (XSS) attack.

Result #1:
Severity: HIGH
State: TO_VERIFY
Status: RECURRENT
Attack Vector:

    1. getInputStream: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[44,49]
    2. xml: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[44,23]
    3. xml: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[47,44]
    4. InputSource: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[47,28]
    5. is: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[47,23]
    6. is: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[48,40]
    7. parse: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[48,39]
    8. doc: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[48,20]
    9. doc: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[49,29]
    10. getDocumentElement: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[49,51]
    11. element: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[49,19]
    12. element: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[50,28]
    13. getChildNodes: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[50,49]
    14. nodes: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[50,20]
    15. nodes: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[53,31]
    16. nodes: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[54,23]
    17. item: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[54,33]
    18. getNodeName: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[54,48]
    19. print: /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java[54,22]
    Review result in Checkmarx One: Reflected_XSS_All_Clients

@cyates-checkmarx cyates-checkmarx changed the title Reflected_XSS_All_Clients @ xxe.java Reflected_XSS_All_Clients @ /src/main/java/org/cysecurity/cspf/jvl/controller/xxe.java Dec 11, 2023
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

1 participant