RDKB-64593 : Coverity issues in ccsp-common-library phase 2

[ksures101]

Reason for change: To address high impact coverity issues
Test Procedure: There should not be any coverity issues mentioned in the ticket
Risks: None

Signed-off-by: kavyachowdahalli_suresh@comcast.com

[Copilot]

Pull request overview

This PR addresses high-impact Coverity findings in ccsp-common-library by tightening control-flow to avoid unsafe operations and ensure allocated resources are released on error paths.

Changes:

• Prevent potential out-of-bounds/overflow behavior in BspTemplateArchiveSaveToFile() by stopping the write loop when sprintf_s fails.
• Fix resource leaks in SetParamValueHelper() by routing error returns through a cleanup label that frees allocated strings.
• Fix resource leaks in SlapDslhParamtoIsParameterReadOnly() by ensuring allocated strings are freed before returning the read-only result.

Reviewed changes

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

File	Description
source/cosa/utilities/bsp_eng/bspeng_archive_co_process.c	Stops comma-separated serialization loop on sprintf_s failure to avoid unsafe subsequent operations.
source/cosa/package/slap/services/dslh/SlapDslhParamTree/slap_dslh_paramto_access.c	Uses goto cleanup on error/return paths to consistently release allocated resources and resolve Coverity leak reports.

---

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