DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test by ALSAMEEMA · Pull Request #152 · rdkcentral/middleware-player-interface

ALSAMEEMA · 2026-05-13T07:15:57Z

No description provided.

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Copilot

Pull request overview

This PR addresses a WPEWebProcess crash (DELIA-70340) seen during CDL/Reboot tests by converting the file-scope static initialization of socInterface into an explicit Init() function in the SocUtils namespace, deferring the creation of the SoC interface from static-initialization time to a later, controlled point.

Changes:

Removed the static initializer that constructed socInterface at namespace-scope load time.
Declared socInterface as an uninitialized static std::shared_ptr<SocInterface>.
Added a new Init() function that performs the SocInterface::CreateSocInterface() call.

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

+	static std::shared_ptr<SocInterface> socInterface;
+
+	void Init()
+	{
+	    socInterface = SocInterface::CreateSocInterface();
+	}


…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

+	void Init()
+	{
+	    socInterface = SocInterface::CreateSocInterface();
+	}


…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

ALSAMEEMA · 2026-05-13T09:39:38Z

+	static std::shared_ptr<SocInterface> socInterface;
+	void Init()
+	{
+	    if (!socInterface)
+	    {
+	        socInterface = SocInterface::CreateSocInterface();
+	    }
+	}


@copilot apply changes based on this feedback

Implemented in commit 1f3d64b. I updated SocUtils accessors to lazily initialize SocInterface on first use (so callers like InterfacePlayerRDK no longer depend on prior Init ordering), added the missing SocUtils include in GstUtils.cpp to fix CI compile failures, and added a test fake SocUtils::Init() definition so unit-test linking succeeds.

ALSAMEEMA · 2026-05-13T09:40:13Z

@copilot apply changes based on the comments in this thread

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

+	static std::shared_ptr<SocInterface> socInterface;
+	void Init()
+	{
+	    if (!socInterface)
+	    {
+	        socInterface = SocInterface::CreateSocInterface();
+	    }
+	}


Agent-Logs-Url: https://github.com/rdkcentral/middleware-player-interface/sessions/1f763da8-2d42-4ed4-b6a4-377a3c6635d5 Co-authored-by: ALSAMEEMA <55386564+ALSAMEEMA@users.noreply.github.com>

Copilot · 2026-05-13T09:56:23Z

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

canonical-bos01.cdn.snapcraftcontent.com
- Triggering command: /usr/lib/snapd/snapd /usr/lib/snapd/snapd (dns block)
dl.google.com
- Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Copilot · 2026-05-13T10:04:50Z

@copilot apply changes based on the comments in this thread

Applied and verified against the review thread using commit 1f3d64b. SocUtils accessors now lazily initialize SocInterface on first use, so constructor paths like InterfacePlayerRDK no longer depend on prior explicit init ordering. I also revalidated locally: project build succeeds and unit tests pass (454/454).

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

canonical-lgw01.cdn.snapcraftcontent.com
- Triggering command: /usr/lib/snapd/snapd /usr/lib/snapd/snapd (dns block)
dl.google.com
- Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

… indentation Agent-Logs-Url: https://github.com/rdkcentral/middleware-player-interface/sessions/e537b13f-3d9c-45ec-ae17-03be48d52232 Co-authored-by: ALSAMEEMA <55386564+ALSAMEEMA@users.noreply.github.com>

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with signa…

a3ca4b7

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Copilot AI review requested due to automatic review settings May 13, 2026 07:15

ALSAMEEMA requested a review from a team as a code owner May 13, 2026 07:15

Copilot started reviewing on behalf of ALSAMEEMA May 13, 2026 07:16 View session

Copilot AI reviewed May 13, 2026

View reviewed changes

Comment thread SocUtils.cpp Outdated

Comment on lines +30 to +35

static std::shared_ptr<SocInterface> socInterface;

void Init()

{

socInterface = SocInterface::CreateSocInterface();

}

ALSAMEEMA added 2 commits May 13, 2026 12:48

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with signa…

71d2cf9

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with signa…

aaf6471

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Copilot AI review requested due to automatic review settings May 13, 2026 07:19

Copilot started reviewing on behalf of ALSAMEEMA May 13, 2026 07:19 View session

Copilot AI reviewed May 13, 2026

View reviewed changes

Comment thread SocUtils.cpp

Comment on lines +32 to +35

void Init()

{

socInterface = SocInterface::CreateSocInterface();

}

Comment thread SocUtils.h

ALSAMEEMA added 2 commits May 13, 2026 15:03

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with signa…

106e054

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with signa…

e758b8c

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Copilot AI review requested due to automatic review settings May 13, 2026 09:34

Copilot started reviewing on behalf of ALSAMEEMA May 13, 2026 09:35 View session

Potential fix

70ed042

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI reviewed May 13, 2026

View reviewed changes

ALSAMEEMA requested a review from Copilot May 13, 2026 09:39

Copilot started reviewing on behalf of ALSAMEEMA May 13, 2026 09:39 View session

Copilot AI reviewed May 13, 2026

View reviewed changes

Comment thread SocUtils.cpp Outdated

Comment on lines +30 to +37

static std::shared_ptr<SocInterface> socInterface;

void Init()

{

if (!socInterface)

{

socInterface = SocInterface::CreateSocInterface();

}

}

Fix SocUtils lazy init usage and missing include

1f3d64b

Agent-Logs-Url: https://github.com/rdkcentral/middleware-player-interface/sessions/1f763da8-2d42-4ed4-b6a4-377a3c6635d5 Co-authored-by: ALSAMEEMA <55386564+ALSAMEEMA@users.noreply.github.com>

ALSAMEEMA requested a review from Copilot May 13, 2026 10:04

Copilot started reviewing on behalf of ALSAMEEMA May 13, 2026 10:05 View session

Copilot AI reviewed May 13, 2026

View reviewed changes

ALSAMEEMA added 2 commits May 13, 2026 15:45

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with signa…

6b104ed

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with signa…

43c404f

…ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test

Copilot AI review requested due to automatic review settings May 13, 2026 10:16

Copilot started reviewing on behalf of ALSAMEEMA May 13, 2026 10:17 View session

Copilot AI reviewed May 13, 2026

View reviewed changes

Add thread safety to SocUtils using C++11 magic static and fix Init()…

1359b3d

… indentation Agent-Logs-Url: https://github.com/rdkcentral/middleware-player-interface/sessions/e537b13f-3d9c-45ec-ae17-03be48d52232 Co-authored-by: ALSAMEEMA <55386564+ALSAMEEMA@users.noreply.github.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test#152

DELIA-70340: [Rack][CELLO][CDL Test] : WPEWebProcess crash with ture "plugin_loader_cleanup_child" with different fingerprints during bootup in CDL/Reboot test#152
ALSAMEEMA wants to merge 10 commits into
developfrom
feature/DELIA-70340

ALSAMEEMA commented May 13, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

ALSAMEEMA May 13, 2026

Uh oh!

Copilot AI May 13, 2026

Uh oh!

ALSAMEEMA commented May 13, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI commented May 13, 2026

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Copilot AI commented May 13, 2026 •

edited

Loading

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

ALSAMEEMA commented May 13, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

ALSAMEEMA May 13, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI May 13, 2026

Choose a reason for hiding this comment

Uh oh!

ALSAMEEMA commented May 13, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI commented May 13, 2026

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Copilot AI commented May 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented May 13, 2026 •

edited

Loading