Merge pull request #216 from guruchandru/sync_crash

Webpa SetForceSync Crash Fix
xmidt-org · Mar 2, 2022 · 1c0ca13 · 1c0ca13
2 parents 5e83668 + 13e9c18
commit 1c0ca13
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/source/WebConfig/cosa_webconfig_dml.c b/source/WebConfig/cosa_webconfig_dml.c
@@ -92,6 +92,7 @@ X_RDK_WebConfig_SetParamStringValue
 	}
 	if( AnscEqualString(ParamName, "ForceSync", TRUE))
 	{
+		int session_status = 0;
 		if(AnscEqualString(strValue, "telemetry", TRUE))
         	{
 			char telemetryUrl[256] = {0};
@@ -103,7 +104,7 @@ X_RDK_WebConfig_SetParamStringValue
 			}		
 		}	
 		/* save update to backup */
-		if(setForceSync(strValue, "", 0) == 1)
+		if(setForceSync(strValue, "", &session_status) == 1)
 		{
 			return TRUE;
 		}

diff --git a/source/WebConfig/cosa_webconfig_internal.c b/source/WebConfig/cosa_webconfig_internal.c
@@ -711,7 +711,7 @@ int setWebConfigParameterValues(parameterValStruct_t *val, int paramCount, char
 				else //pass empty transaction id when Force sync is with empty doc
 				{
 					WebcfgDebug("setWebConfigParameterValues empty setForceSync\n");
-					ret = setForceSync(val[i].parameterValue, "", 0);
+					ret = setForceSync(val[i].parameterValue, "", &session_status);
 				}
 				if(session_status)
 				{