-
Notifications
You must be signed in to change notification settings - Fork 3
Release/1.1.1 #138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release/1.1.1 #138
Changes from all commits
2ce89ed
70e2192
ef949f3
3034d01
cc36a87
07fd812
f7638f0
6b11c5b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
/** | ||
Check failure on line 1 in AVInput/AVInput.cpp
|
||
* If not stated otherwise in this file or this component's LICENSE | ||
* file the following copyright and licenses apply: | ||
* | ||
|
@@ -868,7 +868,30 @@ | |
params["frameRateN"] = 60000; | ||
params["frameRateD"] = 1001; | ||
break; | ||
|
||
case dsVIDEO_FRAMERATE_100: | ||
params["frameRateN"] = 100000; | ||
params["frameRateD"] = 1000; | ||
break; | ||
case dsVIDEO_FRAMERATE_119dot88: | ||
params["frameRateN"] = 120000; | ||
params["frameRateD"] = 1001; | ||
break; | ||
case dsVIDEO_FRAMERATE_120: | ||
params["frameRateN"] = 120000; | ||
params["frameRateD"] = 1000; | ||
break; | ||
case dsVIDEO_FRAMERATE_200: | ||
params["frameRateN"] = 200000; | ||
params["frameRateD"] = 1000; | ||
break; | ||
case dsVIDEO_FRAMERATE_239dot76: | ||
params["frameRateN"] = 240000; | ||
params["frameRateD"] = 1001; | ||
break; | ||
case dsVIDEO_FRAMERATE_240: | ||
params["frameRateN"] = 240000; | ||
params["frameRateD"] = 100; | ||
break; | ||
default: | ||
params["frameRateN"] = 60000; | ||
params["frameRateD"] = 1000; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
/** | ||
Check failure on line 1 in HdmiInput/HdmiInput.cpp
|
||
* If not stated otherwise in this file or this component's LICENSE | ||
* file the following copyright and licenses apply: | ||
* | ||
|
@@ -718,7 +718,30 @@ | |
params["frameRateN"] = 60000; | ||
params["frameRateD"] = 1001; | ||
break; | ||
|
||
case dsVIDEO_FRAMERATE_100: | ||
params["frameRateN"] = 100000; | ||
params["frameRateD"] = 1000; | ||
break; | ||
case dsVIDEO_FRAMERATE_119dot88: | ||
params["frameRateN"] = 120000; | ||
params["frameRateD"] = 1001; | ||
break; | ||
case dsVIDEO_FRAMERATE_120: | ||
params["frameRateN"] = 120000; | ||
params["frameRateD"] = 1000; | ||
break; | ||
case dsVIDEO_FRAMERATE_200: | ||
params["frameRateN"] = 200000; | ||
params["frameRateD"] = 1000; | ||
break; | ||
case dsVIDEO_FRAMERATE_239dot76: | ||
params["frameRateN"] = 240000; | ||
params["frameRateD"] = 1001; | ||
break; | ||
case dsVIDEO_FRAMERATE_240: | ||
params["frameRateN"] = 240000; | ||
params["frameRateD"] = 100; | ||
break; | ||
default: | ||
params["frameRateN"] = 60000; | ||
params["frameRateD"] = 1000; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,7 +17,7 @@ | |
* limitations under the License. | ||
**/ | ||
|
||
#include <gtest/gtest.h> | ||
Check failure on line 20 in Tests/L1Tests/tests/test_HdcpProfile.cpp
|
||
|
||
#include "HdcpProfile.h" | ||
|
||
|
@@ -31,7 +31,6 @@ | |
#include "dsMgr.h" | ||
#include "dsDisplay.h" | ||
#include "ThunderPortability.h" | ||
#include "pwrMgr.h" | ||
#include "PowerManagerMock.h" | ||
|
||
#include <iostream> | ||
|
@@ -505,16 +504,6 @@ | |
return Core::ERROR_NONE; | ||
})); | ||
|
||
ON_CALL(*p_iarmBusImplMock, IARM_Bus_Call) | ||
.WillByDefault( | ||
[](const char* ownerName, const char* methodName, void* arg, size_t argLen) { | ||
if (strcmp(methodName, IARM_BUS_PWRMGR_API_GetPowerState) == 0) { | ||
auto* param = static_cast<IARM_Bus_PWRMgr_GetPowerState_Param_t*>(arg); | ||
param->curState = IARM_BUS_PWRMGR_POWERSTATE_ON; | ||
} | ||
return IARM_RESULT_SUCCESS; | ||
}); | ||
|
||
EVENT_SUBSCRIBE(0, _T("onDisplayConnectionChanged"), _T("client.events"), message); | ||
|
||
IARM_Bus_DSMgr_EventData_t eventData; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
/* | ||
* If not stated otherwise in this file or this component's LICENSE file the | ||
Check failure on line 2 in Tests/L1Tests/tests/test_HdmiCecSource.cpp
|
||
* following copyright and licenses apply: | ||
* | ||
* Copyright 2022 RDK Management | ||
|
@@ -26,7 +26,6 @@ | |
#include "HdmiCecSourceImplementation.h" | ||
#include "HdmiCec.h" | ||
#include "HdmiCecSource.h" | ||
#include "pwrMgr.h" | ||
#include "PowerManagerMock.h" | ||
#include "FactoriesImplementation.h" | ||
#include "IarmBusMock.h" | ||
|
@@ -365,16 +364,6 @@ | |
|
||
return IARM_RESULT_SUCCESS; | ||
})); | ||
|
||
ON_CALL(*p_iarmBusImplMock, IARM_Bus_Call) | ||
.WillByDefault( | ||
[](const char* ownerName, const char* methodName, void* arg, size_t argLen) { | ||
if (strcmp(methodName, IARM_BUS_PWRMGR_API_GetPowerState) == 0) { | ||
auto* param = static_cast<IARM_Bus_PWRMgr_GetPowerState_Param_t*>(arg); | ||
param->curState = IARM_BUS_PWRMGR_POWERSTATE_ON; | ||
} | ||
return IARM_RESULT_SUCCESS; | ||
}); | ||
|
||
} | ||
virtual ~HdmiCecSourceTest() override | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -40,7 +40,7 @@ git clone https://[email protected]/rdkcentral/entservices-testframework. | |
echo "======================================================================================" | ||
echo "buliding thunderTools" | ||
cd ThunderTools | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/Tests/L1Tests/patches/00010-R4.4-Add-support-for-project-dir.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/patches/00010-R4.4-Add-support-for-project-dir.patch | ||
cd - | ||
|
||
|
||
|
@@ -59,10 +59,10 @@ echo "========================================================================== | |
echo "buliding thunder" | ||
|
||
cd Thunder | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/Tests/L2Tests/patches/Use_Legact_Alt_Based_On_ThunderTools_R4.4.3.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/Tests/L2Tests/patches/error_code_R4_4.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/Tests/L1Tests/patches/1004-Add-support-for-project-dir.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/Tests/L1Tests/patches/RDKEMW-733-Add-ENTOS-IDS.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/patches/Use_Legact_Alt_Based_On_ThunderTools_R4.4.3.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/patches/error_code_R4_4.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/patches/1004-Add-support-for-project-dir.patch | ||
patch -p1 < $GITHUB_WORKSPACE/entservices-testframework/patches/RDKEMW-733-Add-ENTOS-IDS.patch | ||
cd - | ||
|
||
cmake -G Ninja -S Thunder -B build/Thunder \ | ||
|
@@ -153,7 +153,6 @@ touch rdk/iarmbus/libIBus.h | |
touch rdk/iarmbus/libIBusDaemon.h | ||
touch rdk/iarmmgrs-hal/deepSleepMgr.h | ||
touch rdk/iarmmgrs-hal/mfrMgr.h | ||
touch rdk/iarmmgrs-hal/pwrMgr.h | ||
touch rdk/iarmmgrs-hal/sysMgr.h | ||
touch network/wifiSrvMgrIarmIf.h | ||
touch network/netsrvmgrIarm.h | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
# If not stated otherwise in this file or this component's license file the | ||
Check failure on line 1 in cmake/FindIARMBus.cmake
|
||
# following copyright and licenses apply: | ||
# | ||
# Copyright 2020 RDK Management | ||
|
@@ -28,11 +28,10 @@ | |
find_library(IARMBUS_LIBRARIES NAMES IARMBus) | ||
find_path(IARMBUS_INCLUDE_DIRS NAMES libIARM.h PATH_SUFFIXES rdk/iarmbus) | ||
find_path(IARMRECEIVER_INCLUDE_DIRS NAMES receiverMgr.h PATH_SUFFIXES rdk/iarmmgrs/receiver) | ||
find_path(IARMPWR_INCLUDE_DIRS NAMES pwrMgr.h PATH_SUFFIXES rdk/iarmmgrs-hal) | ||
|
||
set(IARMBUS_LIBRARIES ${IARMBUS_LIBRARIES} CACHE PATH "Path to IARMBus library") | ||
set(IARMBUS_INCLUDE_DIRS ${IARMBUS_INCLUDE_DIRS} ${IARMRECEIVER_INCLUDE_DIRS} ${IARMPWR_INCLUDE_DIRS}) | ||
set(IARMBUS_INCLUDE_DIRS ${IARMBUS_INCLUDE_DIRS} ${IARMRECEIVER_INCLUDE_DIRS} ${IARMPWR_INCLUDE_DIRS} CACHE PATH "Path to IARMBus include") | ||
set(IARMBUS_INCLUDE_DIRS ${IARMBUS_INCLUDE_DIRS} ${IARMRECEIVER_INCLUDE_DIRS}) | ||
set(IARMBUS_INCLUDE_DIRS ${IARMBUS_INCLUDE_DIRS} ${IARMRECEIVER_INCLUDE_DIRS} CACHE PATH "Path to IARMBus include") | ||
|
||
include(FindPackageHandleStandardArgs) | ||
FIND_PACKAGE_HANDLE_STANDARD_ARGS(IARMBUS DEFAULT_MSG IARMBUS_INCLUDE_DIRS IARMBUS_LIBRARIES) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coverity Issue - Uninitialized pointer field
Non-static class member "pwrMgrEventHandler" is not initialized in this constructor nor in any functions that it calls.
Medium Impact, CWE-457
UNINIT_CTOR