Eyevinn / mp4ff / 28998478488

83%

master: 83%

Build Type

Pull #525

github

Committed by tobbee

Commit Message

refactor(crypto): immutable InitProtectData with per-sequence FragmentEncryptor

The AV1 protection-range function captures a mutable frame-header decoder whose
reference-frame state accumulates across samples in decode order. Storing it in the
long-lived, apparently-shareable InitProtectData (as ProtFunc did) made it unsafe to
reuse across segments or share between concurrent encryptions.

Split immutable config from per-run state: InitProtectData now holds an unexported
sampleProtector factory (no ProtFunc field, ProtectionRangeFunc removed) and is safe to
share. A new FragmentEncryptor (InitProtectData.NewFragmentEncryptor) owns the mutable,
decode-order state, and EncryptFragments encrypts a whole decode sequence with one fresh
protector - the correct entry point for multi-fragment AV1 segments. EncryptFragment
remains for a single self-contained fragment.

HEVC parameter sets are now parsed once per run instead of per sample. mp4ff-encrypt uses
one FragmentEncryptor per media segment. Adds a concurrency test that shares one
InitProtectData across goroutines encrypting AV1 (race-free, identical output).

Pull Request Pull Request #525: refactor(crypto): immutable InitProtectData with per-sequence FragmentEncryptor

Coverage Stats

67 of 98 new or added lines in 2 files covered. (68.37%)

3 existing lines in 2 files now uncovered.

23153 of 27981 relevant lines covered (82.75%)

0.92 hits per line