kollol2k / CxFlowGithub

GNU General Public License v2.0
0 stars 1 forks source link

CX Reflected_XSS_All_Clients @ src/main/java/org/owasp/benchmark/testcode/BenchmarkTest01141.java [master] #316

Closed kollol2k closed 4 years ago

kollol2k commented 4 years ago

Reflected_XSS_All_Clients issue exists @ src/main/java/org/owasp/benchmark/testcode/BenchmarkTest01141.java in branch master

The application's doPost embeds untrusted data in the generated output with println, at line 40 of src\main\java\org\owasp\benchmark\testcode\BenchmarkTest01141.java. This untrusted data is embedded straight into the output without proper sanitization or encoding, enabling an attacker to inject malicious code into the output. The attacker would be able to alter the returned web page by simply providing modified data in the user input getRequestURI, which is read by the doPost method at line 40 of src\main\java\org\owasp\benchmark\testcode\BenchmarkTest01141.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.

Severity: High

CWE:79

Checkmarx

Lines: 99 94


Code (Line #99):

user + " has been remembered with cookie: " + rememberMe.getName() 

Code (Line #94):

                rememberMe.setPath(request.getRequestURI()); // i.e., set path to JUST this servlet