{"id":"CVE-2026-113664","aliases":["GHSA-97wv-p4xx-c7mg"],"title":"ApostropheCMS import-export tar guard misses './..' because it is a literal '../' substring test","summary":"ApostropheCMS import-export tar guard misses './..' because it is a literal '../' substring test","severity":"medium","cvss":5.3,"cwe":["CWE-22"],"vendor":"apostrophecms","product":"@apostrophecms/import-export","ecosystem":"npm","affected":["@apostrophecms/import-export < 3.7.0"],"patched":["@apostrophecms/import-export 3.7.0"],"published":"2026-10-20","updated":"2026-10-20","sourceUpdated":"2026-10-20T18:23:43Z","source":"GHSA","sourceUrl":"https://github.com/advisories/GHSA-97wv-p4xx-c7mg","references":[{"url":"https://github.com/apostrophecms/apostrophe/security/advisories/GHSA-97wv-p4xx-c7mg"},{"url":"https://github.com/apostrophecms/apostrophe/commit/9500b8b0e1d630d5e07fbb5fd2c25ed0943845ce"},{"url":"https://github.com/apostrophecms/apostrophe/commit/f84fd41cbf6c22eb8d0008fc9c25df1f7130b350"},{"url":"https://github.com/apostrophecms/apostrophe/releases/tag/@apostrophecms/import-export@3.7.0"},{"url":"https://github.com/advisories/GHSA-97wv-p4xx-c7mg"}],"tags":["ghsa","npm","cve.org"],"cvssVector":"CVSS:4.0/AV:N/AC:L/AT:N/PR:L/UI:N/VC:L/VI:L/VA:L/SC:N/SI:N/SA:N","cvssSource":"cna","ingestedAt":"2026-10-20T18:14:37.222Z","slug":"CVE-2026-113664","body":"## Overview\n\n### Summary\n\nThe tar-extraction guard added to `@apostrophecms/import-export` for GHSA-mwxc-m426-3f78 is a literal substring test for `../`, and `./..` does not contain that substring. An entry named `./..` therefore passes the guard, `path.join` normalizes it to the parent of the export directory, and the resulting `fs.WriteStream` opens a directory and emits `EISDIR` with no `error` listener attached, which terminates the Node process.\n\nConfirmed on default-branch HEAD `1bd5d0a7a6fe880836fa75adad6964f1c577d640`, package version `3.6.2`.\n\n### Details\n\n`packages/import-export/lib/formats/gzip.js`, in the tar-stream `entry` handler inside `extract(filepath, exportPath)`:\n\n```js\nconst name = header.name.replace(/\\\\/g, '/');\nif (name.includes('../')) {\n  // skipped\n}\n...\nstream.pipe(fs.WriteStream(path.join(exportPath, name)));\n```\n\nTwo separate problems sit in those four lines. The guard tests the raw entry name for a substring instead of resolving the joined path and requiring containment, and the write stream has no `error` handler, so any stream error becomes an unhandled `'error'` event.\n\nReaching it needs an account that can import, nothing more. `packages/import-export/modules/@apostrophecms/import-export-page/index.js` exposes `post.importExportImport`, which calls into `lib/methods/import.js`; that handler requires `req.user` and then calls `format.input(exportPath)`, which for a `.gz` upload is `gzip.js` calling `extract()` on the uploaded archive.\n\n### PoC\n\nThis is not an end-to-end exploit. What follows is each of the three primitives the crash depends on, executed individually, plus a source-level trace of the route. The chain from `post.importExportImport` to `extract()` was read, not observed.\n\nThe guard misses the payload:\n\n```bash\nnode -e \"console.log('./..'.includes('../')); console.log('..'.includes('../')); console.log('a/./..'.includes('../'));\"\n```\n\n```text\nfalse\nfalse\nfalse\n```\n\n`path.join` then resolves it out of the export directory:\n\n```bash\nnode -e \"const path=require('path'); const exportPath='/tmp/exportRoot'; console.log(path.join(exportPath,'./..'));\"\n```\n\n```text\n/tmp\n```\n\nAnd a write stream opened on a directory, with no `error` listener, kills the process rather than failing the request:\n\n```bash\nnode -e \"const fs=require('fs'); const s=fs.createWriteStream('/tmp'); s.write('x'); s.end(); setTimeout(()=>console.log('still alive'),500);\"\n```\n\n```text\nevents.js:291\n      throw er; // Unhandled 'error' event\n      ^\n\nError: EISDIR: illegal operation on a directory, open '/tmp'\nEmitted 'error' event on WriteStream instance at:\n    at internal/fs/streams.js:361:12\n    at FSReqCallback.oncomplete (fs.js:156:23) {\n  errno: -21,\n  code: 'EISDIR',\n  syscall: 'open',\n  path: '/tmp'\n}\n```\n\nThe `events.js:291` frame shows that transcript came off the Node 14 line my audit box had. The unhandled-`error`-event behavior is not version specific, but the exact message on your supported Node versions is worth a re-run.\n\nSource-side confirmation without running anything:\n\n```bash\ngrep -n \"includes('../')\" packages/import-export/lib/formats/gzip.js\ngrep -n \"WriteStream\" packages/import-export/lib/formats/gzip.js\n```\n\n### Impact\n\nAn authenticated user who can reach the import UI uploads one crafted archive and takes down the Node process handling it. Every worker that picks up such an archive dies the same way.\n\n### Suggested fix\n\nResolve the target and require containment, and attach an error handler:\n\n```js\nconst base = path.resolve(exportPath);\nconst target = path.resolve(path.join(exportPath, name));\n\nif (target !== base && !target.startsWith(base + path.sep)) {\n  stream.on('end', next);\n  stream.resume();\n  return;\n}\n\nif (header.type === 'directory') {\n  fsp.mkdir(target).then(next).catch(reject);\n} else {\n  const out = fs.WriteStream(target);\n  out.on('error', reject);\n  stream.pipe(out);\n  stream.on('end', next);\n}\n```\n\nThe error handler earns its place independently. Any tar entry that lands on an existing directory name has the same effect whether or not traversal is involved.\n\n### Why this is not GHSA-mwxc-m426-3f78\n\nThe `name.includes('../')` check is the mitigation shipped for GHSA-mwxc-m426-3f78 / CVE-2026-32731. It stops names carrying the literal `../` sequence. It does not canonicalize the joined path, so `./..` and a bare `..` still get through, and it did not touch the missing `error` listener. This report is about what the substring test leaves behind.\n\n\nAI assistance was used for the code audit and drafting. The exact guard and `WriteStream` lines in `packages/import-export/lib/formats/gzip.js` at the stated HEAD were manually verified, the import route from the page module through `lib/methods/import.js` into the entry handler were manually traced, and the three Node one-liners were manually ran and copied verbatim.\n\n## Affected packages\n\n- `@apostrophecms/import-export < 3.7.0`\n\n## Remediation\n\nUpgrade to a patched release:\n\n- `@apostrophecms/import-export 3.7.0`","depth":"sunlit","depthScore":29,"depthScoreParts":{"impact":29.2,"likelihood":0,"exploitation":0,"ransomware":0},"changes":[]}