kmcdon83 / dvwa Goto Github PK

Second_Order_SQL_Injection issue exists @ dvwa/includes/dvwaPage.inc.php in branch master

Method &dvwaSessionGrab at line 53 of dvwa\includes\dvwaPage.inc.php gets database data from the _SESSION_dvwa element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\captcha\source\medium.php. This may enable an Second-Order SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 57

Code (Line #57):

	return $_SESSION[ 'dvwa' ];

File_Manipulation issue exists @ vulnerabilities/upload/source/high.php in branch master

The input obtained via <?php in the file vulnerabilities\upload\source\high.php at line 1 is used to determine the location of a file to be written into by <?php in the file vulnerabilities\upload\source\high.php at line 1, potentially allowing an attacker to alter or corrupt the contents of that file, or create a new file altogether.

Severity: High

CWE:552

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 12

Code (Line #12):

	$uploaded_tmp  = $_FILES[ 'uploaded' ][ 'tmp_name' ];

Command_Injection issue exists @ vulnerabilities/exec/source/low.php in branch master

The application's <?php method calls an OS (shell) command with shell_exec, at line 1 of vulnerabilities\exec\source\low.php, using an untrusted string with the command to execute.  
This could allow an attacker to inject an arbitrary command, and enable a Command Injection attack.
The attacker may be able to inject the executed command via user input, _REQUEST, which is retrieved by the application in the <?php method, at line 1 of vulnerabilities\exec\source\low.php.

Severity: High

CWE:77

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$target = $_REQUEST[ 'ip' ];

Command_Injection issue exists @ vulnerabilities/exec/source/medium.php in branch master

The application's <?php method calls an OS (shell) command with shell_exec, at line 1 of vulnerabilities\exec\source\medium.php, using an untrusted string with the command to execute.  
This could allow an attacker to inject an arbitrary command, and enable a Command Injection attack.
The attacker may be able to inject the executed command via user input, _REQUEST, which is retrieved by the application in the <?php method, at line 1 of vulnerabilities\exec\source\medium.php.

Severity: High

CWE:77

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$target = $_REQUEST[ 'ip' ];

Second_Order_SQL_Injection issue exists @ vulnerabilities/sqli/source/high.php in branch master

Method <?php at line 1 of vulnerabilities\sqli\source\high.php gets database data from the _SESSION_id element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\sqli\source\high.php. This may enable an Second-Order SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$id = $_SESSION[ 'id' ];

SQL_Injection issue exists @ vulnerabilities/sqli_blind/source/low.php in branch master

Method <?php at line 1 of vulnerabilities\sqli_blind\source\low.php gets user input from the _GET element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\sqli_blind\source\low.php. This may enable an SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$id = $_GET[ 'id' ];

Code_Injection issue exists @ vulnerabilities/view_help.php in branch master

The application's <?php method receives and dynamically executes user-controlled code using eval, at line 1 of vulnerabilities\view_help.php. This could enable an attacker to inject and run arbitrary code. 
The attacker can inject the executed code via user input, _GET, which is retrieved by the application in the <?php method, at line 1 of vulnerabilities\view_help.php.

Severity: High

CWE:94

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 11

Code (Line #11):

$id       = $_GET[ 'id' ];

Command_Injection issue exists @ vulnerabilities/exec/source/impossible.php in branch master

The application's <?php method calls an OS (shell) command with shell_exec, at line 1 of vulnerabilities\exec\source\impossible.php, using an untrusted string with the command to execute.  
This could allow an attacker to inject an arbitrary command, and enable a Command Injection attack.
The attacker may be able to inject the executed command via user input, _REQUEST, which is retrieved by the application in the <?php method, at line 1 of vulnerabilities\exec\source\impossible.php.

Severity: High

CWE:77

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 8

Code (Line #8):

	$target = $_REQUEST[ 'ip' ];

File_Disclosure issue exists @ external/phpids/0.6/lib/IDS/Log/Email.php in branch master

The input obtained via isSpamAttempt in the file external\phpids\0.6\lib\IDS\Log\Email.php at line 210 is used to determine the file to be read by isSpamAttempt in the file external\phpids\0.6\lib\IDS\Log\Email.php at line 210, potentially disclosing the contents of that file.

Severity: High

CWE:538

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 237

Code (Line #237):

        $userAgent  = $_SERVER['HTTP_USER_AGENT'];

File_Manipulation issue exists @ vulnerabilities/upload/source/impossible.php in branch master

The input obtained via <?php in the file vulnerabilities\upload\source\impossible.php at line 1 is used to determine the location of a file to be written into by <?php in the file vulnerabilities\upload\source\impossible.php at line 1, potentially allowing an attacker to alter or corrupt the contents of that file, or create a new file altogether.

Severity: High

CWE:552

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 9

Code (Line #9):

	$uploaded_name = $_FILES[ 'uploaded' ][ 'name' ];

SQL_Injection issue exists @ vulnerabilities/brute/source/low.php in branch master

Method <?php at line 1 of vulnerabilities\brute\source\low.php gets user input from the _GET element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\brute\source\low.php. This may enable an SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$user = $_GET[ 'username' ];

File_Disclosure issue exists @ vulnerabilities/view_source.php in branch master

The input obtained via <?php in the file vulnerabilities\view_source.php at line 1 is used to determine the file to be read by <?php in the file vulnerabilities\view_source.php at line 1, potentially disclosing the contents of that file.

Severity: High

CWE:538

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 11 12

Code (Line #11):

$id       = $_GET[ 'id' ];

Code (Line #12):

$security = $_GET[ 'security' ];

SQL_Injection issue exists @ vulnerabilities/sqli_blind/source/impossible.php in branch master

Method <?php at line 1 of vulnerabilities\sqli_blind\source\impossible.php gets user input from the _GET element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\sqli_blind\source\impossible.php. This may enable an SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 8

Code (Line #8):

	$id = $_GET[ 'id' ];

SQL_Injection issue exists @ vulnerabilities/sqli/source/impossible.php in branch master

Method <?php at line 1 of vulnerabilities\sqli\source\impossible.php gets user input from the _GET element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\sqli\source\impossible.php. This may enable an SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 8

Code (Line #8):

	$id = $_GET[ 'id' ];

File_Manipulation issue exists @ external/phpids/0.6/lib/IDS/Log/Email.php in branch master

The input obtained via isSpamAttempt in the file external\phpids\0.6\lib\IDS\Log\Email.php at line 210 is used to determine the location of a file to be written into by isSpamAttempt in the file external\phpids\0.6\lib\IDS\Log\Email.php at line 210, potentially allowing an attacker to alter or corrupt the contents of that file, or create a new file altogether.

Severity: High

CWE:552

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 237

Code (Line #237):

        $userAgent  = $_SERVER['HTTP_USER_AGENT'];

Reflected_XSS_All_Clients issue exists @ vulnerabilities/sqli/session-input.php in branch master

Method <?php at line 1 of vulnerabilities\sqli\session-input.php gets user input for the _POST element. This element’s value then flows through the code without being properly sanitized or validated and is eventually displayed to the user in method dvwaSourceHtmlEcho at line 382 of dvwa\includes\dvwaPage.inc.php. This may enable a Cross-Site-Scripting attack.

Severity: High

CWE:79

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 12

Code (Line #12):

	$_SESSION[ 'id' ] =  $_POST[ 'id' ];

Reflected_XSS_All_Clients issue exists @ vulnerabilities/view_source.php in branch master

Method <?php at line 1 of vulnerabilities\view_source.php gets user input for the _GET element. This element’s value then flows through the code without being properly sanitized or validated and is eventually displayed to the user in method dvwaSourceHtmlEcho at line 382 of dvwa\includes\dvwaPage.inc.php. This may enable a Cross-Site-Scripting attack.

Severity: High

CWE:79

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 11 12

Code (Line #11):

$id       = $_GET[ 'id' ];

Code (Line #12):

$security = $_GET[ 'security' ];

Command_Injection issue exists @ vulnerabilities/exec/source/high.php in branch master

The application's <?php method calls an OS (shell) command with shell_exec, at line 1 of vulnerabilities\exec\source\high.php, using an untrusted string with the command to execute.  
This could allow an attacker to inject an arbitrary command, and enable a Command Injection attack.
The attacker may be able to inject the executed command via user input, _REQUEST, which is retrieved by the application in the <?php method, at line 1 of vulnerabilities\exec\source\high.php.

Severity: High

CWE:77

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$target = trim($_REQUEST[ 'ip' ]);

SQL_Injection issue exists @ vulnerabilities/sqli_blind/source/high.php in branch master

Method <?php at line 1 of vulnerabilities\sqli_blind\source\high.php gets user input from the _COOKIE element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\sqli_blind\source\high.php. This may enable an SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$id = $_COOKIE[ 'id' ];

SQL_Injection issue exists @ dvwa/includes/DBMS/PGSQL.php in branch master

Method <?php at line 1 of dvwa\includes\DBMS\PGSQL.php gets user input from the _SERVER element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of dvwa\includes\DBMS\PGSQL.php. This may enable an SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 56

Code (Line #56):

$baseUrl = 'http://'.$_SERVER[ 'SERVER_NAME' ].$_SERVER[ 'PHP_SELF' ];

SQL_Injection issue exists @ vulnerabilities/sqli/source/low.php in branch master

Method <?php at line 1 of vulnerabilities\sqli\source\low.php gets user input from the _REQUEST element. This element’s value then flows through the code without being properly sanitized or validated, and is eventually used in a database query in method <?php at line 1 of vulnerabilities\sqli\source\low.php. This may enable an SQL Injection attack.

Severity: High

CWE:89

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$id = $_REQUEST[ 'id' ];

Stored_XSS issue exists @ dvwa/includes/dvwaPage.inc.php in branch master

Method dvwaGuestbook at line 503 of dvwa\includes\dvwaPage.inc.php gets data from the database, for the mysqli_fetch_row element. This element’s value then flows through the code without being properly filtered or encoded and is eventually displayed to the user in method dvwaHtmlEcho at line 183 of dvwa\includes\dvwaPage.inc.php. This may enable a Stored Cross-Site-Scripting attack.

Severity: High

CWE:79

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 509

Code (Line #509):

	while( $row = mysqli_fetch_row( $result ) ) {

File_Disclosure issue exists @ vulnerabilities/view_source_all.php in branch master

The input obtained via <?php in the file vulnerabilities\view_source_all.php at line 1 is used to determine the file to be read by <?php in the file vulnerabilities\view_source_all.php at line 1, potentially disclosing the contents of that file.

Severity: High

CWE:538

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 11

Code (Line #11):

$id = $_GET[ 'id' ];

File_Manipulation issue exists @ external/phpids/0.6/lib/IDS/Log/File.php in branch master

The input obtained via prepareData in the file external\phpids\0.6\lib\IDS\Log\File.php at line 146 is used to determine the location of a file to be written into by execute in the file external\phpids\0.6\lib\IDS\Log\File.php at line 177, potentially allowing an attacker to alter or corrupt the contents of that file, or create a new file altogether.

Severity: High

CWE:552

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 163

Code (Line #163):

                              urlencode($_SERVER['REQUEST_URI']),

Reflected_XSS_All_Clients issue exists @ vulnerabilities/csp/source/jsonp.php in branch master

Method <?php at line 1 of vulnerabilities\csp\source\jsonp.php gets user input for the _GET element. This element’s value then flows through the code without being properly sanitized or validated and is eventually displayed to the user in method <?php at line 1 of vulnerabilities\csp\source\jsonp.php. This may enable a Cross-Site-Scripting attack.

Severity: High

CWE:79

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 5

Code (Line #5):

	$callback = $_GET['callback'];

File_Disclosure issue exists @ vulnerabilities/view_help.php in branch master

The input obtained via <?php in the file vulnerabilities\view_help.php at line 1 is used to determine the file to be read by <?php in the file vulnerabilities\view_help.php at line 1, potentially disclosing the contents of that file.

Severity: High

CWE:538

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 11

Code (Line #11):

$id       = $_GET[ 'id' ];

Reflected_XSS_All_Clients issue exists @ dvwa/includes/dvwaPage.inc.php in branch master

Method dvwaSecurityLevelGet at line 135 of dvwa\includes\dvwaPage.inc.php gets user input for the _COOKIE element. This element’s value then flows through the code without being properly sanitized or validated and is eventually displayed to the user in method dvwaHtmlEcho at line 183 of dvwa\includes\dvwaPage.inc.php. This may enable a Cross-Site-Scripting attack.

Severity: High

CWE:79

Vulnerability details and guidance

Internal Guidance

Checkmarx

Training
Recommended Fix

Lines: 112 136

Code (Line #112):

	dvwaRedirect( $_SERVER[ 'PHP_SELF' ] );

Code (Line #136):

	return isset( $_COOKIE[ 'security' ] ) ? $_COOKIE[ 'security' ] : 'impossible';