RDKCOM-5420: TELEMETRY-2 Dynamic telemetry markers for Dynamic Tables in Telemetry

[onkar-panchare1]

Description:
Introduced marker type DataModelTable for dynamic tables in telemetry.

Enhanced T2 agent to resolve and collect data from all instances of multi-instance object tables when dynamic markers are configured.

Added logic to parse DataModelTable, filter markers based on telemetry profile configuration, and encode them in telemetry reports after collection, ensuring telemetry reports include per-instance data for configured dynamic markers.

Added support for an "index" parameter in configuration to restrict data collection to specific indexes if required.

Reason for change:
Enable telemetry to capture dynamic, per-instance data from tables (e.g., Host table, Wi-Fi associated devices) where indexes are not fixed, ensuring richer and more flexible telemetry collection.

[CLAassistant]

All committers have signed the CLA.

[onkar-panchare1]

@shibu-kv Kindly help with more details on coverity scan failed

[shibu-kv]

@onkar-panchare1 Thank you for the contribution.
Could you please help with incorporating below suggestions :
1] The report profile schemas are available in the repo here - https://github.com/rdkcentral/telemetry/tree/develop/schemas
Could you please amend the schemas that aligns with the new capabilities supported in the report profile configuration.
2] This project maintains an integration test suite as well in location - https://github.com/rdkcentral/telemetry/tree/develop/test/functional-tests
Could you please help to add tests to add coverage for this new feature addition as well.
3] To maintain a uniform formatting we do enforce a common formatter based on linux utility astyle.
Please format this contribution by running below commands in your fork :
find . -name '.c' -o -name '.h' | xargs astyle --options=.astylerc
find . -name '*.orig' -type f -delete

Formatter configs are already on the project - https://github.com/rdkcentral/telemetry/blob/develop/.astylerc

[shibu-kv]

Requesting to incorporate few changes outlined ingeneric comments.

[onkar-panchare1]

Requesting to incorporate few changes outlined ingeneric comments.

Hi @shibu-kv
I have updated the MR with uniform formatting and have updated the schema to support new capabilities.
I will be adding tests soon, the MR can be reviewed.

[shibu-kv]

Changes looks good. Waiting for the CI workflows results.

[rdkcmf-jenkins]

Coverity Issue - Data race condition

Accessing "profile->cachedReportList" without holding lock "_Profile.reuseThreadMutex". Elsewhere, "_Profile.cachedReportList" is written to with "_Profile.reuseThreadMutex" held 5 out of 7 times.

Medium Impact, CWE-366
MISSING_LOCK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
source/bulkdata/profile.c:1460

[rdkcmf-jenkins]

Coverity Issue - Data race condition

Accessing "profile->grepSeekProfile->execCounter" without holding lock "plMutex". Elsewhere, "_GrepSeekProfile.execCounter" is written to with "plMutex" held 2 out of 2 times.

Medium Impact, CWE-366
MISSING_LOCK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
source/bulkdata/profile.c:349

[rdkcmf-jenkins]

Coverity Issue - Use after free

Passing freed pointer "header" as an argument to "T2Log".

High Impact, CWE-416
USE_AFTER_FREE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
source/t2parser/t2parser.c:1343

[rdkcmf-jenkins]

Coverity Issue - Free of array-typed value

"free" frees incorrect pointer "content".

High Impact, CWE-590
BAD_FREE

[rdkcmf-jenkins]

Coverity Issue - Free of array-typed value

"free" frees incorrect pointer "content".

High Impact, CWE-590
BAD_FREE

[shibu-kv]

By passing and merging with below known tech debts :

L1 unit tests are failing due to critical bugs:

NULL Pointer Dereference in profile.c (CRASH - highest priority)
Memory Leaks in reportgen.c (2 error paths)
Buffer Overflow Risk in reportgen.c (unbounded strcat)
Error Handling Issues in t2parser.c (continues with invalid data)
USE_AFTER_FREE in t2parser.c (Coverity HIGH severity)

Merging so that team an work on addressing the issues from latest and greatest code from develop. Not all dev groups are having access to update contributors' fork.