Bug 218182 – [GStreamer][EME][Thunder] Do not sanitize CENC init data

• Component: New Bugs
• Status: RESOLVED INVALID

[Xabier Rodríguez Calvar]

[GStreamer][EME][Thunder] Do not sanitize CENC init data

[Xabier Rodríguez Calvar]

Created attachment 412311 [details]
Patch

[Xabier Rodríguez Calvar]

Created attachment 412322 [details]
Patch

[Philippe Normand]

Comment on attachment 412322 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=412322&action=review

> Source/WebCore/platform/encryptedmedia/CDMPrivate.h:77
> +    WEBCORE_EXPORT virtual RefPtr<SharedBuffer> sanitizeInitData(const AtomString& initDataType, const SharedBuffer& initData) const;

Why the WEBCORE_EXPORT?

[Xabier Rodríguez Calvar]

Because the former patch failed to build without it when linking with the Mock in the tests.

[Philippe Normand]

Failed how?

[Xabier Rodríguez Calvar]

Link error cause the mock is in the tests and that is not in WebCore

[Xabier Rodríguez Calvar]

Committed r269103: <https://trac.webkit.org/changeset/269103>

[Radar WebKit Bug Importer]

<rdar://problem/70764876>

[Philippe Normand]

I'm afraid this actually goes against the spec:

https://www.w3.org/TR/encrypted-media/#dom-mediakeysession-generaterequest (step 10.2 quoted below)

---
Let sanitized init data be a validated and sanitized version of init data.

The user agent must thoroughly validate the Initialization Data before passing it to the CDM. This includes verifying that the length and values of fields are reasonable, verifying that values are within reasonable limits, and stripping irrelevant, unsupported, or unknown data or fields. It is recommended that user agents pre-parse, sanitize, and/or generate a fully sanitized version of the Initialization Data. If the Initialization Data format specified by initDataType supports multiple entries, the user agent should remove entries that are not needed by the CDM. The user agent must not re-order entries within the Initialization Data.
---

I think the real issue is in the app side. Let's revert this patch?

[WebKit Commit Bot]

Re-opened since this is blocked by bug 233630