LibreKAT/Ocideck
LibreKAT/Ocideck
Watch
4
0
Fork
You've already forked Ocideck
 0
Code Issues 2 Pull requests Projects Releases 4 Packages Activity Actions

bug(export): pas de afbeeldingsdimensiegrens ook toe op GIF in HTML-export #1044

New issue
Closed
opened 2026-08-01 08:26:04 +00:00 by brenno · 1 comment
brenno commented 2026-08-01 08:26:04 +00:00
Owner
Copy link

Context
encodeForHtmlEmbed laat GIF en SVG vóór de dimensieprobe ongewijzigd door. Voor SVG is dat logisch, maar een kleine geldige GIF kan een extreem groot canvas hebben. De gewone beelddecoder begrenst op 4096px; de HTML-export omzeilt die grens.

Impact
Een toegestane GIF tot 64 MiB met bijvoorbeeld 30000×30000 pixels kan bij openen van de HTML-export meerdere gigabytes decodegeheugen per frame vragen.

Relevante code

  • lib/services/html_image_embedder.dart:28 en 55
  • lib/services/export_service.dart:337

Acceptatie

  • Rasterformaten, inclusief GIF, passeren vóór insluiten een betrouwbare dimensiecontrole.
  • Oversized GIF wordt geweigerd of veilig naar een begrensde representatie omgezet.
  • Regressietest gebruikt een kleine GIF-header met canvas boven kMaxImageDecodeDimension en bewijst dat hij niet als ongewijzigde data-URI wordt ingesloten.
Context encodeForHtmlEmbed laat GIF en SVG vóór de dimensieprobe ongewijzigd door. Voor SVG is dat logisch, maar een kleine geldige GIF kan een extreem groot canvas hebben. De gewone beelddecoder begrenst op 4096px; de HTML-export omzeilt die grens. Impact Een toegestane GIF tot 64 MiB met bijvoorbeeld 30000×30000 pixels kan bij openen van de HTML-export meerdere gigabytes decodegeheugen per frame vragen. Relevante code - lib/services/html_image_embedder.dart:28 en 55 - lib/services/export_service.dart:337 Acceptatie - Rasterformaten, inclusief GIF, passeren vóór insluiten een betrouwbare dimensiecontrole. - Oversized GIF wordt geweigerd of veilig naar een begrensde representatie omgezet. - Regressietest gebruikt een kleine GIF-header met canvas boven kMaxImageDecodeDimension en bewijst dat hij niet als ongewijzigde data-URI wordt ingesloten.
brenno commented 2026-08-01 19:38:17 +00:00
Author
Owner
Copy link

Opgelost op main (PR #1088, merge in 4713374b). GIF loopt nu langs een inhouds-gebaseerde dimensieprobe vóór insluiten: binnen kMaxImageDecodeDimension ongewijzigd mee (beweging behouden), daarboven geweigerd. Regressietest met een 30000x30000-GIF-header.

Opgelost op main (PR #1088, merge in 4713374b). GIF loopt nu langs een inhouds-gebaseerde dimensieprobe vóór insluiten: binnen kMaxImageDecodeDimension ongewijzigd mee (beweging behouden), daarboven geweigerd. Regressietest met een 30000x30000-GIF-header.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/markdown-editor-gate-followup-1123
feat/native-calls-f2-xmpp
codex/markdown-editor-translations
codex/markdown-editor-ux
codex/template-followup
cockpit-wip
v0.1.3
v0.1.2
v0.1.1
v0.1.0
archive/git-mirror
Labels
Clear labels   
accepted
Assessed and will be built

  
bug
Something is broken

  
declined
Assessed and will not be built

  
docs
Documentation only

  
duplicate
Already tracked elsewhere

  
enhancement
New feature or improvement request

  
good first issue
Small, self-contained, and a good place to start

  
in-progress
Someone is working on this right now. Do not start it in parallel.

  
needs-info
Waiting for the reporter

  
privacy
Personal data can leak. Handled before anything else.

  
security
A weakness someone could use. Handled before anything else.

  
triage
Awaiting assessment

No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
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
LibreKAT/Ocideck#1044
No description provided.