Skip to content

RDKEMW-19892: [RDKEMW] [BCM Rogers Monarch IUI V2] Linear trick play (FF/RW/Seek) leads to blank screen, device hang, followed by automatic reboot. Multiple critical processes (gmem, WPENetworkProce, WorkerPoolType: etc.) crash were observed.#197

Merged
sankarimuthu merged 1 commit into
support/0.2.0_8.6from
feature/RDKEMW-19892_8.6
Jul 21, 2026

Conversation

@ALSAMEEMA

Copy link
Copy Markdown

No description provided.

…eads to blank screen, device hang, followed by automatic reboot. Multiple critical processes (gmem, WPENetworkProce, WorkerPoolType: etc.) crash were observed.
Copilot AI review requested due to automatic review settings July 10, 2026 09:43
@ALSAMEEMA
ALSAMEEMA requested a review from a team as a code owner July 10, 2026 09:43
@ALSAMEEMA
ALSAMEEMA changed the base branch from main to support/0.2.0_8.6 July 10, 2026 09:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates Widevine DRM key selection logic to better handle default key ID matching (including UUID string formats) and to avoid invalid map indexing that can lead to incorrect key selection during trick-play operations.

Changes:

  • Add UUID (dashed string) → 16-byte binary conversion so cencData can be matched against binary Key IDs parsed from PSSH.
  • Make getKey() use map-safe lookup (find/begin) instead of assuming contiguous integer slots (e.g., at(0)).
  • Improve fallback behavior and logging when the requested key slot is not found.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread drm/helper/WidevineDrmHelper.cpp
Comment thread drm/helper/WidevineDrmHelper.cpp
@sankarimuthu
sankarimuthu merged commit 9eebe0b into support/0.2.0_8.6 Jul 21, 2026
12 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants