Fix memory leak in DownloadRuntimeFeatutres from early return paths

[Copilot]

The DownloadRuntimeFeatutres function allocated hashParam and its strdup'd members (hashvalue, hashtime) but had multiple early return paths that bypassed cleanup, leaking memory on cert selector failures and MTLS cert fetch failures.

Changes

• Refactored to single exit point using goto cleanup pattern
• Initialize hashParam members to nullptr immediately after allocation for safe cleanup
• Replace early returns (cert selector init failure, MTLS failures, URL validation) with goto cleanup
• Added unified cleanup section that safely frees all allocated memory on all code paths
• Removed inline cleanup code that only executed on success path

Before

if (thisCertSel == NULL) {
    RDK_LOG(RDK_LOG_DEBUG, LOG_RFCMGR, "Cert selector initialization failed\n");
    return cert_ret_code;  // Leaks hashParam and its strdup'd members
}

After

if (thisCertSel == NULL) {
    RDK_LOG(RDK_LOG_DEBUG, LOG_RFCMGR, "Cert selector initialization failed\n");
    ret_value = cert_ret_code;
    goto cleanup;  // Ensures proper cleanup
}

cleanup:
    if (hashParam != nullptr) {
        if (hashParam->hashvalue != nullptr) free(hashParam->hashvalue);
        if (hashParam->hashtime != nullptr) free(hashParam->hashtime);
        free(hashParam);
    }
    return ret_value;

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.