Skip to content
go-libfossil Check-in [38eddf43f1]
Login

Check-in [38eddf43f1]

Overview
Comment:fix(delta): refuse a source or target of 2^32 bytes or more in Create (#279) (#320)

The delta format holds integers below 2^32 only, so a producer must not write a target length or copy offset at or past that. Create now returns ErrTooLarge for such inputs; Deltify then stores the content whole, as fossil does when a delta is not worthwhile, and the other callers propagate the error. Apply's header check becomes >= so it refuses exactly what Create will not write. The bound is tested on lengths, as a direct test needs a 4 GiB input.

Timelines: family | ancestors | descendants | both | main | trunk
Files: files | file ages | folders
SHA3-256: 38eddf43f1b521a58bba1515653b71c769f145366be2a3361c04c56e987ac4db
User & Date: [email protected] on 2026-10-11 23:34:08.000
Other Links: branch diff | manifest | tags
Context
2026-10-11
23:39
fix(manifest): refuse check-in filenames outside the writer profile (#270) (#321)

Checkin wrote any F-card filename it was given. The artifact format's writer profile (4.6.1) forbids a backslash and anything but valid UTF-8, surrogates and overlong forms included; fossil rejects a manifest that names one. storeFileBlobs now refuses such a name or old name before storing anything. check-in: 357d10dac6 user: [email protected] tags: main, trunk

23:34
fix(delta): refuse a source or target of 2^32 bytes or more in Create (#279) (#320)

The delta format holds integers below 2^32 only, so a producer must not write a target length or copy offset at or past that. Create now returns ErrTooLarge for such inputs; Deltify then stores the content whole, as fossil does when a delta is not worthwhile, and the other callers propagate the error. Apply's header check becomes >= so it refuses exactly what Create will not write. The bound is tested on lengths, as a direct test needs a 4 GiB input. check-in: 38eddf43f1 user: [email protected] tags: main, trunk

23:28
fix(blob): fill a phantom in place when its content is stored (#278) (#319)

blob.Store, StoreDelta and StoreDeltaRaw now share writeRow, which fills a phantom row of the same name in place (content, size, and removal from the phantom table) as fossil's content_put does, and returns that rid. existingName no longer treats a phantom as stored content, so a check-in of a file matching a known phantom stores the file instead of recording a manifest that points at nothing. check-in: 23cfc00bfe user: [email protected] tags: main, trunk

Changes