Skip to content

Commit 8aeb187

Browse files
leenaS-darun-madhavan-013tkathi259
authored andcommitted
RDKEMW-4220: Fix wpeframework crash on reactivating plugin and powerstate change (#104)
* RDKEMW-4220: Fix wpeframework crash on reactivating plugin and powerstate change Signed-off-by: leenaS-d <[email protected]> * RDKEMW-4220 : Fix wpeframework crash on reactivating HdmiCecSource plugin and launch cobalt/htmlapp/webkitbrowser Reason for changes: The service unregister call is missed during HdmiCecSource plugin deactivation. Test Procedure: Verified build, tested. --------- Signed-off-by: leenaS-d <[email protected]> Co-authored-by: Arun P Madhavan <[email protected]> Co-authored-by: tkathi259 <[email protected]>
1 parent 45a2946 commit 8aeb187

File tree

3 files changed

+3
-0
lines changed

3 files changed

+3
-0
lines changed

HdmiCecSink/HdmiCecSink.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -821,6 +821,7 @@ namespace WPEFramework
821821
{
822822
if(_powerManagerPlugin)
823823
{
824+
_powerManagerPlugin->Unregister(_pwrMgrNotification.baseInterface<Exchange::IPowerManager::IModeChangedNotification>());
824825
_powerManagerPlugin.Reset();
825826
}
826827
_registeredEventHandlers = false;

HdmiCecSource/HdmiCecSource.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,7 @@ namespace WPEFramework
150150
}
151151

152152
_connectionId = 0;
153+
_service->Unregister(&_notification);
153154
_service->Release();
154155
_service = nullptr;
155156
LOGINFO("HdmiCecSource plugin is deactivated. Successfully deactivated HdmiCecSource Plugin");

HdmiCecSource/HdmiCecSourceImplementation.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -385,6 +385,7 @@ namespace WPEFramework
385385

386386
if(_powerManagerPlugin)
387387
{
388+
_powerManagerPlugin->Unregister(_pwrMgrNotification.baseInterface<Exchange::IPowerManager::IModeChangedNotification>());
388389
_powerManagerPlugin.Reset();
389390
}
390391
_registeredEventHandlers = false;

0 commit comments

Comments
 (0)