Read 2026-11-03
sumatrapdfreader/sumatrapdf: technical due diligence, read from its repository
What Borehole read in sumatrapdfreader/sumatrapdf's code and history at fcde16b: its licence, security advisories, releases, review practice, tests, dependencies and operations. Every finding cites the file and line it rests on. The full report also covers the team and asks the questions the code cannot answer.
| Dimension | Rating |
|---|---|
| Architecture & Codebase — including AI provenance | strong |
| Engineering Process & SDLC Maturity | adequate |
| Product & Engineering Maturity | strong |
| Security & Reliability Posture | strong |
| Observability & Operations | not assessable |
| Fit with the Acquirer | not assessable |
What the repository says
- Releases are cut from another branch than the one surveyed
COPYING: Relicensed: GPL-2.0 → GPL-3.0 (2009-07-03)
Architecture & Codebase — including AI provenance
strongext/a-unrar/unrar.cpp is 29,278 lines, 172× this codebase's median file
adequateThe median source file here is 170 lines. 284 files exceed 1,700 lines, the largest being 29,278. Size is not a defect on its own — some problems genuinely live in one place — but these are where merge conflicts, review fatigue and single-owner knowledge concentrate, and they are the first thing that slows an inheriting engineer down.
ext/a-unrar/unrar.cpp:1 — 29,278 lines (172× median) |
ext/a-libwebp/libwebp.c:1 — 24,890 lines (146× median) |
ext/mupdf/source/pdf/cmaps/Adobe-Japan1-UCS2.h:1 — 19,899 lines (117× median) |
ext/heicdec/heic.c:1 — 19,381 lines (114× median) |
ext/tesseract/src/ccutil/universalambigs.h:1 — 19,049 lines (112× median) |
Engineering Process & SDLC Maturity
adequateLittle of the recent change carries a trace of review
adequateIn the last 24 months, 201 of 7,640 commits (2.6%) carry any trace of review: a merge, a pull-request reference, a Reviewed-by or Reviewed-on trailer, a Closes/Fixes reference, a maintainer's sign-off on someone else's change, or a change applied by someone other than its author. That is low but not absent. Review may happen in a workflow this history does not record; a buyer should ask how change is approved before it ships.
| 201 of 7,640 commits in the last 24 months carry review evidence; 2.0% across all 23,504 |
Product & Engineering Maturity
strongReleases are cut from another branch than the one surveyed
adequateThe newest release, 3.4.3, is not on the branch this survey read; the newest release on it is 3.2. The shipped product may differ from what this report describes. Survey the release branch too.
| tag 3.4.3 |
Security & Reliability Posture
strong12 dependencies float, and no lockfile is committed
adequateDependency versions are declared as ranges and no lockfile is committed, so a build today does not necessarily produce what a build last month produced. That matters twice over in a transaction: the thing being bought cannot be reproduced exactly, and a compromised upstream release arrives without anyone choosing to take it.
cmd/package.json:3 — ^1.3.10 |
cmd/package.json:4 — ^25.5.0 |
cmd/package.json:5 — ^8.0.7 |
ext/mupdf/docs/requirements.txt:1 — any version |
ext/mupdf/docs/requirements.txt:2 — any version |
3 shell commands built from strings
adequate3 places run a command through a shell (`shell=True`, `os.system`, `exec` with a template) rather than passing arguments. Where any part comes from outside, it is command injection. Ask where each one's input comes from.
ext/mupdf/setup.py:191 — command run through a shell |
ext/mupdf/setup.py:279 — command run through a shell |
ext/mupdf/setup.py:335 — command run through a shell |
Relicensed: GPL-2.0 → GPL-3.0 (2009-07-03)
adequateThe root licence changed 1 time(s). Code taken under earlier terms stays under them; check which terms any fork or customer relied on.
24 advisories against this project in the last 24 months, 11 high or critical
adequateThe newest was published on 2026-10-27. A project that publishes advisories is telling its users about its fixes, and the count and pace say how much security work it carries. Whether 8 of them are fixed in the code you ship could not be told from this history: its version is only known to be at least 3.2.0. Answer lives in: the release you run, set against each fixed version. GHSA-27p6-wrfp-pgm8 (moderate), GHSA-jf4v-rw66-j4w2 (high), GHSA-5mwx-65x7-cffp (moderate) and 1 more are fixed in the surveyed commit; which release carries them cannot be told from this history, which does not place the release users run. Answer lives in: the version you run. The record is incomplete: NVD: 7 records name sumatrapdf with nothing tying them to this repository (CVE-2009-1605, CVE-2009-4117, CVE-2012-4895, …).
| 2026-10-27: GHSA-5wm7-225j-9qcj (high): SumatraPDF: Unauthenticated WM_COPYDATA/DDE CmdExec Channel Allows Arbitrary Local Program Execution and Restricted-Mode Policy Bypass |
| 2026-10-26: GHSA-m2fg-gc9h-qv9c (high), fixed in djvudec 5925e18 (github.com/kjk/djvudec): Heap buffer overflow in bundled DjVu decoder (djvudec) via integer overflow of JB2 bitmap dimensions |
| 2026-09-24: CVE-2026-88361 (high): SumatraPDF 3.6.1 contains an integer overflow vulnerability in EngineMupdf::BuildPageLabelRec() when parsing PDF PageLabels /Nums entries. |
| 2026-09-16: GHSA-86hq-m6hv-67h2 (moderate): Unsigned-to-signed hdrLen validation bypass in SumatraPDF MOBI parsing causes out-of-bounds read |
| 2026-09-16: GHSA-vfpj-qhvj-92wg (moderate): Signed integer overflow in SumatraPDF LIT header parsing causes invalid-pointer read |
Observability & Operations
not assessableNot rated: only 4 of 16 checks could reach a conclusion here; the rest do not apply to this repository or could not read it.
Fit with the Acquirer
not assessableNot rated: structurally not assessable from a repository.
Read more, or read your own
The full report on sumatrapdfreader/sumatrapdf: every dimension, the questions for the room, and the audit log.
Survey a repository: free on public repositories. Private code is read on your own machine by a collector whose source you can read first.