jesse-a/OpenCRM
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases 1 Packages Wiki Activity Actions

v0.12 — Input validation: CSV-formula, zip-slip, mass-assignment (3 sub-fixes) #109

New issue
Closed
opened 2026-05-26 18:23:41 +00:00 by jesse-a · 1 comment
jesse-a commented 2026-05-26 18:23:41 +00:00
Owner
Copy link

Severity: MEDIUM (v0.12 — pentest-rapport)

  1. CSV formula-injectie — toCsv() prefixt cellen die beginnen met = + - @ \t \r met een apostrof; voorkomt dat Excel/LibreOffice ze als formule evalueert (kan exfiltratie via WEBSERVICE/HYPERLINK).
  2. Zip-slip — bestandsnaam-sanitization in saveUploadedFile() + dubbele slug() op receiptOriginal in boekhouder-ZIP-paden; voorkomt path-traversal via geprepareerde archive-entries.
  3. Mass-assignment hardening — Deal.ownerId gevalideerd tegen actieve User; Invoice.status enum-validatie. Voorkomt dat een POST met extra fields willekeurige columns kan zetten.
**Severity: MEDIUM** (v0.12 — pentest-rapport) 1. **CSV formula-injectie** — `toCsv()` prefixt cellen die beginnen met `= + - @ \t \r` met een apostrof; voorkomt dat Excel/LibreOffice ze als formule evalueert (kan exfiltratie via WEBSERVICE/HYPERLINK). 2. **Zip-slip** — bestandsnaam-sanitization in `saveUploadedFile()` + dubbele `slug()` op `receiptOriginal` in boekhouder-ZIP-paden; voorkomt path-traversal via geprepareerde archive-entries. 3. **Mass-assignment hardening** — `Deal.ownerId` gevalideerd tegen actieve User; `Invoice.status` enum-validatie. Voorkomt dat een POST met extra fields willekeurige columns kan zetten.
jesse-a commented 2026-05-26 18:23:42 +00:00
Author
Owner
Copy link

Opgelost in commit aa68ac1.

Opgelost in commit aa68ac1.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
v1.0.0
v0.11
v0.10
v0.9
v0.8
v0.7
v0.6
v0.5
Labels
Clear labels   
deferred
Known issue, deferred

  
fixed
Resolved

  
security
Security review finding

  
severity-high
High severity

  
severity-low
Low severity

  
severity-medium
Medium severity

No labels
deferred
fixed
security
severity-high
severity-low
severity-medium
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jesse-a/OpenCRM#109
No description provided.