Skip to content
This repository was archived by the owner on Jan 26, 2023. It is now read-only.
/ rdk-aamp Public archive

Commit cf462cd

Browse files
asujam192Paulpandian Mariappan
asujam192
authored and
Paulpandian Mariappan
committed
DELIA-58846:[Auto]Error logs are printing for aamp config params that are not set using RFC
Reason for change: When RFC was not set unessory error logs printed, Condition to handle valid case of RFC not being set was missing. Fix added to handle this by adding trace log when no RFC is set. Test Procedure: Test steps are on ticket Risks: Low Signed-off-by: Amritha Suja Mohan <[email protected]> Change-Id: Ib9a9a28b410827f668cf8561f3443f315113b40f (cherry picked from commit b313247)
1 parent 1039dbd commit cf462cd

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

AampConfig.cpp

+2-3
Original file line numberDiff line numberDiff line change
@@ -1751,9 +1751,8 @@ void AampConfig::ReadAllTR181Params()
17511751
}
17521752
}
17531753
else
1754-
{
1755-
AAMPLOG_ERR("ReadAllTR181Params: Got emptry value for param:%s",strParamName.c_str());
1756-
1754+
{ // normal case - no need to log by default
1755+
AAMPLOG_TRACE("ReadAllTR181Params: Got empty value for param:%s",strParamName.c_str());
17571756
}
17581757
}
17591758
}

AampRfc.cpp

+5
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,11 @@ namespace RFCSettings
5252
AAMPLOG_INFO("RFC Parameter for %s is %s type = %d", parameter.c_str(), param.value, param.type);
5353
strhost = std::string(param.value);
5454
}
55+
else if (tr181ValueIsEmpty == status)
56+
{
57+
// NO RFC is set , which is success case
58+
AAMPLOG_TRACE("RFC Parameter : %s is not set", parameter.c_str());
59+
}
5560
else
5661
{
5762
AAMPLOG_ERR("get RFC Parameter for %s Failed : %s type = %d", parameter.c_str(), getTR181ErrorString(status), param.type);

0 commit comments

Comments
 (0)