GHSA-PHHP-9RM9-6GR2
Vulnerability from github – Published: 2026-04-08 15:03 – Updated: 2026-04-08 15:03Summary
A malicious note synced to another user can trigger remote code execution in the SiYuan Electron desktop client. The root cause is that table caption content is stored without safe escaping and later unescaped into rendered HTML, creating a stored XSS sink. Because the desktop renderer runs with nodeIntegration enabled and contextIsolation disabled, attacker-controlled JavaScript executes with access to Node.js APIs. In practice, an attacker can import a crafted note into a synced workspace, wait for the victim to sync, and achieve code execution when the victim opens the note.
Details
The vulnerability exists in the table caption handling path. When a table block is parsed, the caption attribute is saved into the node's IAL properties without proper HTML escaping. Later, during rendering, that value is read back, passed through HTML unescaping, and written directly into the output DOM. This turns an attacker-controlled caption into active HTML inside the rendered note.
I confirmed that a crafted table caption containing encoded HTML such as <img src=x onerror=...> is rendered as a live DOM element instead of inert text. This makes the issue a stored XSS. I also confirmed that the most practical delivery path is not Markdown import, but a crafted .sy.zip note imported into a synced workspace. Once synced to another desktop client, opening the note executes the payload automatically.
In the Electron desktop client, this XSS results in code execution rather than browser-only script execution. The renderer is configured with nodeIntegration: true and contextIsolation: false, so JavaScript running in the note context can call Node.js APIs directly. A payload such as require('child_process').exec('calc') executes successfully, demonstrating code execution on the victim machine in the context of the logged-in user.
PoC
- SiYuan Desktop Client A: attacker
- SiYuan Desktop Client B: victim
- Both clients are configured to use the same sync target
PoC File
I created a malicious .sy.zip note containing a table block with a crafted caption property.
Safe validation payload:
<img src=x onerror=alert('caption-xss')>
RCE validation payload on Windows:
<img src=x onerror=require('child_process').exec('calc')>
Steps to Reproduce
1.On Client A, import the crafted .sy.zip note using:
Import -> SiYuan .sy.zip
2.Confirm the imported note appears in the workspace.
3.Trigger sync on Client A so the malicious note is uploaded to the shared sync target.
4.On Client B, trigger sync so the note is downloaded from the shared sync target.
5.Open the synced note on Client B.
Observed Result
With the safe payload, JavaScript executes automatically when the victim opens the note. With the RCE payload, the Electron renderer executes:
require('child_process').exec('calc')
This launches Calculator on Windows, demonstrating code execution in the victim user's context.
Impact
- Impact Across All Platforms: Stored XSS
- Electron Desktop App: Remote Code Execution
{
"affected": [
{
"package": {
"ecosystem": "Go",
"name": "github.com/siyuan-note/siyuan/kernel"
},
"ranges": [
{
"events": [
{
"introduced": "0"
},
{
"fixed": "0.0.0-20260407035653-2f416e5253f1"
}
],
"type": "ECOSYSTEM"
}
]
}
],
"aliases": [
"CVE-2026-39846"
],
"database_specific": {
"cwe_ids": [
"CWE-79",
"CWE-94"
],
"github_reviewed": true,
"github_reviewed_at": "2026-04-08T15:03:54Z",
"nvd_published_at": "2026-04-07T22:16:23Z",
"severity": "CRITICAL"
},
"details": "### Summary\nA malicious note synced to another user can trigger remote code execution in the SiYuan Electron desktop client. The root cause is that table caption content is stored without safe escaping and later unescaped into rendered HTML, creating a stored XSS sink. Because the desktop renderer runs with `nodeIntegration` enabled and `contextIsolation` disabled, attacker-controlled JavaScript executes with access to Node.js APIs. In practice, an attacker can import a crafted note into a synced workspace, wait for the victim to sync, and achieve code execution when the victim opens the note.\n\n### Details\nThe vulnerability exists in the table caption handling path. When a table block is parsed, the `caption` attribute is saved into the node\u0027s IAL properties without proper HTML escaping. Later, during rendering, that value is read back, passed through HTML unescaping, and written directly into the output DOM. This turns an attacker-controlled caption into active HTML inside the rendered note.\n\nI confirmed that a crafted table caption containing encoded HTML such as `\u0026lt;img src=x onerror=...\u0026gt;` is rendered as a live DOM element instead of inert text. This makes the issue a stored XSS. I also confirmed that the most practical delivery path is not Markdown import, but a crafted `.sy.zip` note imported into a synced workspace. Once synced to another desktop client, opening the note executes the payload automatically.\n\nIn the Electron desktop client, this XSS results in code execution rather than browser-only script execution. The renderer is configured with `nodeIntegration: true` and `contextIsolation: false`, so JavaScript running in the note context can call Node.js APIs directly. A payload such as `require(\u0027child_process\u0027).exec(\u0027calc\u0027)` executes successfully, demonstrating code execution on the victim machine in the context of the logged-in user.\n\n### PoC\n\n- SiYuan Desktop Client A: attacker\n- SiYuan Desktop Client B: victim\n- Both clients are configured to use the same sync target\n\n### PoC File\n\nI created a malicious `.sy.zip` note containing a table block with a crafted `caption` property.\n\nSafe validation payload:\n```html\n\u0026lt;img src=x onerror=alert(\u0027caption-xss\u0027)\u0026gt;\n```\nRCE validation payload on Windows:\n```html\n\u0026lt;img src=x onerror=require(\u0027child_process\u0027).exec(\u0027calc\u0027)\u0026gt;\n```\n\n### Steps to Reproduce\n1.On Client A, import the crafted `.sy.zip` note using:\n\n`Import -\u003e SiYuan .sy.zip`\n\n2.Confirm the imported note appears in the workspace.\n\n3.Trigger sync on Client A so the malicious note is uploaded to the shared sync target.\n\n4.On Client B, trigger sync so the note is downloaded from the shared sync target.\n\n5.Open the synced note on Client B.\n\n### Observed Result\nWith the safe payload, JavaScript executes automatically when the victim opens the note.\nWith the RCE payload, the Electron renderer executes:\n```js\nrequire(\u0027child_process\u0027).exec(\u0027calc\u0027)\n```\nThis launches Calculator on Windows, demonstrating code execution in the victim user\u0027s context.\n\n### Impact\n- Impact Across All Platforms: Stored XSS\n- Electron Desktop App: Remote Code Execution",
"id": "GHSA-phhp-9rm9-6gr2",
"modified": "2026-04-08T15:03:55Z",
"published": "2026-04-08T15:03:54Z",
"references": [
{
"type": "WEB",
"url": "https://github.com/siyuan-note/siyuan/security/advisories/GHSA-phhp-9rm9-6gr2"
},
{
"type": "ADVISORY",
"url": "https://nvd.nist.gov/vuln/detail/CVE-2026-39846"
},
{
"type": "PACKAGE",
"url": "https://github.com/siyuan-note/siyuan"
}
],
"schema_version": "1.4.0",
"severity": [
{
"score": "CVSS:3.1/AV:N/AC:L/PR:L/UI:R/S:C/C:H/I:H/A:H",
"type": "CVSS_V3"
}
],
"summary": "SiYuan: Remote Code Execution in the Electron desktop client via stored XSS in synced table captions"
}
Sightings
| Author | Source | Type | Date |
|---|
Nomenclature
- Seen: The vulnerability was mentioned, discussed, or observed by the user.
- Confirmed: The vulnerability has been validated from an analyst's perspective.
- Published Proof of Concept: A public proof of concept is available for this vulnerability.
- Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
- Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
- Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
- Not confirmed: The user expressed doubt about the validity of the vulnerability.
- Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.