| Titre | AdithyaYelloju Restaurant-Management-System 7f0e7e84255e8fcfd488e83f8f91451bbbff6b9c SQL Injection |
|---|
| Description | The affected components are the unauthenticated action scripts under `admin/` that read a record id from the query string and concatenate it directly into a SQL statement:
- `admin/delete1.php` — `mysqli_query($conn,"delete from tables where id='$id'");`
- `admin/delete1_m.php` — `mysqli_query($conn,"delete from menu where id='$id'");`
- `admin/delete_user1.php` — `mysqli_query($conn,"delete from user where id='$id'");`
- `admin/approve.php` — `mysqli_query($conn,"update order_list set status='yes' where id='$id'")`
- `admin/approvet.php` — `mysqli_query($conn,"update t_order set status='Approved' where id='$id'")`
- `admin/notapprove.php` — `mysqli_query($conn,"update order_list set status='no' where id='$id'")`
- `admin/notapprovet.php` — `mysqli_query($conn,"update t_order set status='Cancel' where id='$id'")`
The id is taken directly from `$_GET["id"]` and concatenated into the SQL string with single-quote interpolation. There is no session check, no role check, no anti-CSRF token, and no escaping. An attacker can close the quote, append `OR id>0 -- ` (or `OR '1'='1' -- `) to mass-update or mass-delete rows. Verified in the lab by deleting all `menu` rows through `admin/delete1_m.php`.
|
|---|
| La source | ⚠️ https://github.com/AdithyaYelloju/Restaurant-Management-System/issues/5 |
|---|
| Utilisateur | 100001001X (UID 100909) |
|---|
| Soumission | 01/09/2026 13:26 (il y a 29 jours) |
|---|
| Modérer | 30/09/2026 10:09 (29 days later) |
|---|
| Statut | Accepté |
|---|
| Entrée VulDB | 411922 [AdithyaYelloju Restaurant-Management-System Unauthenticated Action Script admin/delete1.php mysqli_query id injection SQL] |
|---|
| Points | 20 |
|---|