Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for senc and IV size #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zdanek
Copy link

@zdanek zdanek commented Oct 21, 2021

Hi.
I've added support for senc box.

I suppose that there was no support for this box as it depends on IV size that is specified in tenc box that sometimes is not present. For example for segments without Init segment. I've added support for saving IV size set in tenc if it's present.

If IV size is not present in preceding tenc box, reading senc is based on heuristics. It's size can be 0 (cbcs encoding) 8 or 16 bytes. It can guess that it should be 0 bytes by fitting encryption data size * sample_count against provided box size (len).
It works as supposed. I've checked it with different segments.

I'm open for suggestions.

@zdanek
Copy link
Author

zdanek commented Oct 21, 2021

BTW temporarily I've put my version of this code at https://mlynoteka.mlyn.org/mp4parser/
I've fixed Samples for test option as original that refers dashif.org simply does not work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant