Skip to content

Commit

Permalink
fix(HLS): Ban unsupported combinations of SAMPLE-AES encryption (#6295)
Browse files Browse the repository at this point in the history
  • Loading branch information
avelad authored and joeyparrish committed Feb 27, 2024
1 parent f96895b commit 3b60303
Show file tree
Hide file tree
Showing 2 changed files with 44 additions and 1 deletion.
17 changes: 16 additions & 1 deletion lib/hls/hls_parser.js
Expand Up @@ -3382,7 +3382,22 @@ shaka.hls.HlsParser = class {
* @return {?shaka.extern.DrmInfo}
* @private
*/
static identityDrmParser_(drmTag) {
static identityDrmParser_(drmTag, mimeType) {
if (mimeType == 'video/mp2t') {
throw new shaka.util.Error(
shaka.util.Error.Severity.CRITICAL,
shaka.util.Error.Category.MANIFEST,
shaka.util.Error.Code.HLS_MSE_ENCRYPTED_MP2T_NOT_SUPPORTED);
}

if (shaka.util.Platform.isMediaKeysPolyfilled()) {
throw new shaka.util.Error(
shaka.util.Error.Severity.CRITICAL,
shaka.util.Error.Category.MANIFEST,
shaka.util.Error.Code
.HLS_MSE_ENCRYPTED_LEGACY_APPLE_MEDIA_KEYS_NOT_SUPPORTED);
}

const method = drmTag.getRequiredAttrValue('METHOD');
const VALID_METHODS = ['SAMPLE-AES', 'SAMPLE-AES-CTR'];
if (!VALID_METHODS.includes(method)) {
Expand Down
28 changes: 28 additions & 0 deletions test/hls/hls_parser_unit.js
Expand Up @@ -3743,6 +3743,34 @@ describe('HlsParser', () => {
await verifyError(master, media, error, /* onCreateSegmentIndex= */ true);
});

it('if SAMPLE-AES encryption with MSE and mp2t content', async () => {
const master = [
'#EXTM3U\n',
'#EXT-X-STREAM-INF:BANDWIDTH=200,CODECS="avc1",',
'RESOLUTION=960x540,FRAME-RATE=60\n',
'video\n',
].join('');

const media = [
'#EXTM3U\n',
'#EXT-X-TARGETDURATION:6\n',
'#EXT-X-PLAYLIST-TYPE:VOD\n',
'#EXT-X-KEY:METHOD=SAMPLE-AES,',
'URI="fake",\n',
'#EXTINF:5,\n',
'#EXT-X-BYTERANGE:121090@616\n',
'main.ts',
].join('');

const error = new shaka.util.Error(
shaka.util.Error.Severity.CRITICAL,
shaka.util.Error.Category.MANIFEST,
Code.HLS_MSE_ENCRYPTED_MP2T_NOT_SUPPORTED);

await verifyError(master, media, error, /* onCreateSegmentIndex= */ true);
});


describe('if required tags are missing', () => {
/**
* @param {string} master
Expand Down

0 comments on commit 3b60303

Please sign in to comment.