Skip to content

seekable_format: Reset frame state after failed seeks - #4940

Open
terrelln wants to merge 1 commit into
meta:devfrom
terrelln:2026-11-12-seekable-failed-seek-retry
Open

terrelln wants to merge 1 commit into
meta:devfrom
terrelln:2026-11-12-seekable-failed-seek-retry

Conversation

@terrelln

Copy link
Copy Markdown
Contributor

Invalidate the cached frame before seeking, and record the new frame only after its input and decoder state are initialized. Otherwise a failed seek can leave a new object with a NULL input pointer, and retrying within the same frame bypasses initialization.

Add a malformed 65-byte archive regression covering in-memory and custom sources, repeated failed reads, and recovery to a valid frame.

@meta-cla meta-cla Bot added the CLA Signed label Nov 12, 2026
@terrelln
terrelln force-pushed the 2026-11-12-seekable-failed-seek-retry branch from bed67e6 to f98e66f Compare November 12, 2026 18:41
Invalidate the cached frame before seeking, and record the new frame only after its input and decoder state are initialized. Otherwise a failed seek can leave a new object with a NULL input pointer, and retrying within the same frame bypasses initialization.

Add a malformed 65-byte archive regression covering in-memory and custom sources, repeated failed reads, and recovery to a valid frame.
@terrelln
terrelln force-pushed the 2026-11-12-seekable-failed-seek-retry branch from f98e66f to d555811 Compare November 12, 2026 20:40

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants