Summary
YesWiki through the latest development branch contains a SQL injection vulnerability in ReactionManager::deleteUserReaction() that allows any authenticated user to inject arbitrary SQL via the {idreaction} and {id} URL path parameters. The parameters are concatenated directly into a SQL LIKE clause without escaping or parameterization.
This is a sibling of CVE-2026-46670 (unauthenticated SQLi in FormManager::create()). Both share the same root cause — raw string concatenation into SQL queries — but exist in different components.
Root Cause
includes/controllers/ApiController.php line 726:
/**
* @Route("/api/reactions/{idreaction}/{id}/{page}/{username}", methods={"DELETE"}, options={"acl":{"+"}})
*/
ACL "+" = any authenticated user. Parameters flow into ReactionManager::deleteUserReaction() → TripleStore::delete() with raw string concatenation into SQL LIKE clause (line 356).
The if branch (lines 340-354) properly uses $this->dbService->escape(). The else branch does not — the developer applied escaping to one code path but not the other.
PoC
DELETE /wiki/?api/reactions/x%27%20OR%201=1%20OR%20value%20LIKE%20%27/test/SomePage/attacker
Host: localhost:8085
Cookie: <session cookie>
Time-based blind variant via {id} parameter for data exfiltration.
Impact
Full database read/write. Any self-registered user can extract yeswiki_users password hashes and emails.
Suggested Fix
Apply $this->dbService->escape() to all parameters in the else branch, matching the if branch pattern. Also audit all TripleStore::delete() callers that pass $extraSQL.
Credits
Kai Aizen / SnailSploit
References
Summary
YesWiki through the latest development branch contains a SQL injection vulnerability in
ReactionManager::deleteUserReaction()that allows any authenticated user to inject arbitrary SQL via the{idreaction}and{id}URL path parameters. The parameters are concatenated directly into a SQL LIKE clause without escaping or parameterization.This is a sibling of CVE-2026-46670 (unauthenticated SQLi in
FormManager::create()). Both share the same root cause — raw string concatenation into SQL queries — but exist in different components.Root Cause
includes/controllers/ApiController.phpline 726:ACL
"+"= any authenticated user. Parameters flow intoReactionManager::deleteUserReaction()→TripleStore::delete()with raw string concatenation into SQL LIKE clause (line 356).The
ifbranch (lines 340-354) properly uses$this->dbService->escape(). Theelsebranch does not — the developer applied escaping to one code path but not the other.PoC
Time-based blind variant via
{id}parameter for data exfiltration.Impact
Full database read/write. Any self-registered user can extract
yeswiki_userspassword hashes and emails.Suggested Fix
Apply
$this->dbService->escape()to all parameters in theelsebranch, matching theifbranch pattern. Also audit allTripleStore::delete()callers that pass$extraSQL.Credits
Kai Aizen / SnailSploit
References