62336: Fix Coverity defects (1 file) by rdkdevpilot · Pull Request #433 · rdkcentral/telemetry

rdkdevpilot · 2026-09-14T16:31:56Z

#Fix for Coverity Defects

Triggered by: ThamimRazith AbbasAli

Fixed Files

File	Line(s)	CIDs	Checker Types	Description
`source/bulkdata/profilexconf.c`	L538	64949	MISSING_LOCK	Data race condition

Fix Summaries

source/bulkdata/profilexconf.c

The fix correctly addresses the MISSING_LOCK defect by acquiring xconfProfileLock before reading the shared globals isOnDemandReport and isAbortTriggered, copying them to local variables, resetting isAbortTriggered under the lock, then releasing the lock before using the local copies. The logic is semantically preserved: the abort flag reset still occurs once per loop iteration, and the publish status calls use the same values as before. The fix is minimal and introduces no new defects.

Repository Guidelines Applied

.github/copilot-instructions.md (instruction)
.github/instructions/c-embedded.instructions.md (instruction)

Activated skills: thread-safety-analyzer

Defect ID: a9153b55-435d-4580-9dfa-558bdc89f783

Copilot

🟢 Approval recommended

The race fix is addressed; the remaining regression-test request is non-blocking.

Pull request overview

This PR fixes a Coverity-reported data race in XCONF report status handling.

Changes:

Protects shared status flags with xconfProfileLock.
Snapshots and resets flags under the mutex.
Publishes status using local values after unlocking.

File summaries

File	Description
`source/bulkdata/profilexconf.c`	Synchronizes shared report-status state.

Review details

Suppressed comments (1)

source/bulkdata/profilexconf.c:540

This changes the synchronization and status snapshot on the report-thread path, but the only test that exercises ProfileXConf_notifyTimeout(..., true) and verifies publishReportUploadStatus is commented out in source/test/bulkdata/profilexconfTest.cpp:551-595. Please enable or replace it with a focused test that covers the report/status path, including concurrent notification, so this race fix has an active regression check.

        pthread_mutex_lock(&xconfProfileLock);
        bool localIsOnDemandReport = isOnDemandReport;
        bool localIsAbortTriggered = isAbortTriggered;

Files reviewed: 1/1 changed files
Comments generated: 0
Review effort level: Lite

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

github-actions · 2026-09-14T16:43:11Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |21.4%   1077|71.0%    31|    -      0
bulkdata/profilexconf.c                   |45.1%    505|81.2%    16|    -      0
bulkdata/reportprofiles.c                 |57.6%    585| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.8%    298| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |40.4%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.9%    919| 100%    21|    -      0
scheduler/scheduler.c                     |73.6%    371| 100%    12|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |58.9%    185|66.7%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|60.5%  10170|89.9%   347|    -      0

Copilot

Copilot review overview

🔵 Needs a closer look

The on-demand state can still be overwritten, and active regression coverage is missing.

Review effort: Lite
Findings: None

Copilot

Copilot review overview

🟡 Changes recommended

Add or enable regression tests covering locked state snapshots, reset behavior, and abort handling.

Review effort: Lite
Findings: 1

Open (1)

Add regression test for ProfileXConf notification race fix · New

github-actions · 2026-09-30T14:51:06Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |27.0%   1130|73.5%    34|    -      0
bulkdata/profilexconf.c                   |45.5%    519|82.4%    17|    -      0
bulkdata/reportprofiles.c                 |57.5%    586| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.8%    298| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |40.4%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.7%    924| 100%    21|    -      0
scheduler/scheduler.c                     |70.2%    413| 100%    13|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |63.8%    185|73.3%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|60.9%  10285|90.3%   352|    -      0

…into topic/62336

Copilot

Copilot review overview

🟡 Changes recommended

The upload decision still reads isAbortTriggered outside the lock, and test coverage does not fully verify snapshot/reset behavior.

Review effort: Lite
Findings: 1

Open (1)

Test abort reset behavior and non-on-demand failure handling · New

Resolved since last review (1)

Add regression test for ProfileXConf notification race fix

github-actions · 2026-09-30T15:20:56Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |27.0%   1130|73.5%    34|    -      0
bulkdata/profilexconf.c                   |48.1%    526|84.2%    19|    -      0
bulkdata/reportprofiles.c                 |57.5%    586| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.8%    298| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |40.4%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.7%    924| 100%    21|    -      0
scheduler/scheduler.c                     |70.2%    413| 100%    13|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |63.8%    185|73.3%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|61.0%  10292|90.4%   354|    -      0

Sets abort state once. Verifies the first failure publishes ABORTED. Calls the helper again without resetting globals. Verifies the next failure publishes FAILURE, proving isAbortTriggered was reset. Adds a separate non-on-demand test verifying no status is published.

Copilot

Copilot review overview

🟡 Changes recommended

Address the test-state cleanup and missing success-path coverage before approval.

Review effort: Lite
Findings: 1

Open (1)

Reset isOnDemandReport after test execution · New

Resolved since last review (1)

Test abort reset behavior and non-on-demand failure handling

github-actions · 2026-09-30T15:45:27Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |27.0%   1130|73.5%    34|    -      0
bulkdata/profilexconf.c                   |48.3%    528|85.0%    20|    -      0
bulkdata/reportprofiles.c                 |57.5%    586| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.8%    298| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |40.4%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.7%    924| 100%    21|    -      0
scheduler/scheduler.c                     |70.2%    413| 100%    13|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |63.8%    185|73.3%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|61.0%  10294|90.4%   355|    -      0

…test execution' Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Copilot review overview

🔵 Needs a closer look

An unprotected shared-state read remains, and key production and success paths lack test coverage.

Review effort: Lite
Findings: None

Resolved since last review (1)

Reset isOnDemandReport after test execution

Previously missed (1)

In code that hasn't changed since last review

Tests bypass production status call and omit success path

source/bulkdata/profilexconf.c:574

The added tests call the test-only test_publishReportUploadStatusForResult wrapper directly, so the changed production call at this site is never exercised; a regression that removes or misplaces this call would still pass. The success path is also not covered because both tests pass T2ERROR_FAILURE. Please add an active test that drives ProfileXConf_notifyTimeout/the report worker through this call and asserts the success status as well as the abort/reset cases.

Copilot

Copilot review overview

🟢 Approval recommended

No unresolved review issues remain.

Review effort: Lite
Findings: None

github-actions · 2026-09-30T16:50:41Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |27.0%   1130|73.5%    34|    -      0
bulkdata/profilexconf.c                   |48.3%    528|85.0%    20|    -      0
bulkdata/reportprofiles.c                 |57.5%    586| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.8%    298| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |40.4%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.7%    924| 100%    21|    -      0
scheduler/scheduler.c                     |70.2%    413| 100%    13|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |63.8%    185|73.3%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|61.0%  10294|90.4%   355|    -      0

github-actions · 2026-09-30T16:54:22Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |27.0%   1130|73.5%    34|    -      0
bulkdata/profilexconf.c                   |48.3%    528|85.0%    20|    -      0
bulkdata/reportprofiles.c                 |57.5%    586| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.8%    298| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |40.4%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.7%    924| 100%    21|    -      0
scheduler/scheduler.c                     |70.2%    413| 100%    13|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |63.8%    185|73.3%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|61.0%  10294|90.4%   355|    -      0

Copilot

Copilot review overview

🔵 Needs a closer look

The production abort-flag read remains unsynchronized, and tests do not cover the active success-report path.

Review effort: Lite
Findings: None

github-actions · 2026-09-30T18:46:09Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |27.0%   1130|73.5%    34|    -      0
bulkdata/profilexconf.c                   |48.3%    528|85.0%    20|    -      0
bulkdata/reportprofiles.c                 |57.5%    586| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.8%    298| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |46.2%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.7%    924| 100%    21|    -      0
scheduler/scheduler.c                     |70.2%    413| 100%    13|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |63.8%    185|73.3%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|61.2%  10294|90.4%   355|    -      0

Copilot

Copilot review overview

🟡 Changes recommended

Add coverage for the success branch and the production call site.

Review effort: Lite
Findings: 1

Open (1)

Add test coverage for successful report upload status · New

+    }
+    else
+    {
+        publishReportUploadStatus("SUCCESS");


github-actions · 2026-09-30T19:06:00Z

Code Coverage Summary

                                         |Lines       |Functions  |Branches    
Filename                                  |Rate     Num|Rate    Num|Rate     Num
================================================================================
[/mnt/L1_CONTAINER_SHARED_VOLUME/source/]
bulkdata/datamodel.c                      |72.5%    218|88.9%     9|    -      0
bulkdata/profile.c                        |27.0%   1130|73.5%    34|    -      0
bulkdata/profilexconf.c                   |48.3%    528|85.0%    20|    -      0
bulkdata/reportprofiles.c                 |57.5%    586| 100%    27|    -      0
bulkdata/t2eventreceiver.c                |64.1%    301| 100%    10|    -      0
bulkdata/t2markers.c                      |80.5%    169|83.3%    12|    -      0
ccspinterface/busInterface.c              |86.8%     53|85.7%     7|    -      0
ccspinterface/rbusInterface.c             |74.3%    946|97.2%    36|    -      0
commonlib/telemetry_busmessage_sender.c   |40.4%    396|84.2%    19|    -      0
dcautil/dca.c                             |71.7%    674| 100%    20|    -      0
dcautil/dcaproc.c                         |84.3%    230| 100%     7|    -      0
dcautil/dcautil.c                         |78.9%    185| 100%     6|    -      0
dcautil/legacyutils.c                     |79.0%    105| 100%     7|    -      0
privacycontrol/rdkservices_privacyutils.c | 100%     11| 100%     2|    -      0
protocol/http/curlinterface.c             | 100%     24| 100%     3|    -      0
protocol/http/multicurlinterface.c        |59.7%    375|80.0%    10|    -      0
protocol/rbusMethod/rbusmethodinterface.c |60.0%    105|50.0%     6|    -      0
reportgen/reportgen.c                     |55.7%    924| 100%    21|    -      0
scheduler/scheduler.c                     |70.2%    413| 100%    13|    -      0
t2parser/t2parser.c                       |59.4%   1939|88.5%    26|    -      0
t2parser/t2parserxconf.c                  |89.7%    195| 100%     3|    -      0
utils/persistence.c                       |78.7%    253| 100%    10|    -      0
utils/t2MtlsUtils.c                       |82.6%     46|75.0%     4|    -      0
utils/t2collection.c                      |89.6%    201| 100%    18|    -      0
utils/t2common.c                          |63.8%    185|73.3%    15|    -      0
utils/t2log_wrapper.c                     |69.2%     26|50.0%     2|    -      0
utils/vector.c                            |94.9%     79| 100%     8|    -      0
================================================================================
                                   Total:|61.0%  10297|90.4%   355|    -      0

Fix MISSING_LOCK in profilexconf.c

c0ac431

Defect ID: a9153b55-435d-4580-9dfa-558bdc89f783

Copilot AI lite review requested due to automatic review settings September 14, 2026 16:31

rdkdevpilot requested a review from a team as a code owner September 14, 2026 16:31

rdkdevpilot added automated-fix missing_lock labels Sep 14, 2026

Copilot started reviewing on behalf of rdkdevpilot September 14, 2026 16:32 View session

Copilot AI reviewed Sep 14, 2026

View reviewed changes

Merge branch 'develop' into topic/62336

fced432

Copilot AI lite review requested due to automatic review settings September 29, 2026 23:37

Copilot started reviewing on behalf of rdkdevpilot September 29, 2026 23:38 View session

Copilot AI reviewed Sep 29, 2026

View reviewed changes

Merge branch 'develop' into topic/62336

07ec236

Copilot AI lite review requested due to automatic review settings September 30, 2026 14:40

Copilot started reviewing on behalf of rdkdevpilot September 30, 2026 14:41 View session

Copilot AI reviewed Sep 30, 2026

View reviewed changes

Comment thread source/bulkdata/profilexconf.c Outdated

tabbas651 added 2 commits September 30, 2026 11:09

Added ProfileXConf_OnDemandStatusSnapshot test cases

cdfcc8e

Merge branch 'topic/62336' of https://github.com/rdkcentral/telemetry …

c7e0899

…into topic/62336

Copilot AI lite review requested due to automatic review settings September 30, 2026 15:10

Copilot started reviewing on behalf of rdkdevpilot September 30, 2026 15:11 View session

Copilot AI reviewed Sep 30, 2026

View reviewed changes

Comment thread source/test/bulkdata/profilexconfTest.cpp Outdated

Copilot AI lite review requested due to automatic review settings September 30, 2026 15:34

Copilot started reviewing on behalf of rdkdevpilot September 30, 2026 15:35 View session

Copilot AI reviewed Sep 30, 2026

View reviewed changes

Comment thread source/test/bulkdata/profilexconfTest.cpp

Potential fix for pull request finding 'Reset isOnDemandReport after …

7e812ab

…test execution' Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI lite review requested due to automatic review settings September 30, 2026 16:41

Copilot started reviewing on behalf of rdkdevpilot September 30, 2026 16:42 View session

Update profilexconfTest.cpp

51dea24

Copilot AI reviewed Sep 30, 2026

View reviewed changes

Copilot AI lite review requested due to automatic review settings September 30, 2026 16:45

Copilot started reviewing on behalf of rdkdevpilot September 30, 2026 16:46 View session

Copilot AI reviewed Sep 30, 2026

View reviewed changes

Update L2-tests.yml

05dd4f4

Copilot AI lite review requested due to automatic review settings September 30, 2026 18:37

Copilot started reviewing on behalf of rdkdevpilot September 30, 2026 18:37 View session

Copilot AI reviewed Sep 30, 2026

View reviewed changes

Merge branch 'develop' into topic/62336

45dc4f4

Copilot AI lite review requested due to automatic review settings September 30, 2026 18:55

Copilot started reviewing on behalf of rdkdevpilot September 30, 2026 18:56 View session

Copilot AI reviewed Sep 30, 2026

View reviewed changes

Comment thread source/bulkdata/profilexconf.c

}

else

{

publishReportUploadStatus("SUCCESS");

shibu-kv approved these changes Sep 30, 2026

View reviewed changes

shibu-kv merged commit 2de35cf into develop Sep 30, 2026
14 of 16 checks passed

github-actions Bot locked and limited conversation to collaborators Sep 30, 2026

Conversation

rdkdevpilot commented Sep 14, 2026 • edited by tabbas651 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Fixed Files

Fix Summaries

Repository Guidelines Applied

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟢 Approval recommended

Suppressed comments (1)

Uh oh!

github-actions Bot commented Sep 14, 2026

Code Coverage Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🔵 Needs a closer look

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

github-actions Bot commented Sep 30, 2026

Code Coverage Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

github-actions Bot commented Sep 30, 2026

Code Coverage Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

github-actions Bot commented Sep 30, 2026

Code Coverage Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🔵 Needs a closer look

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟢 Approval recommended

Uh oh!

github-actions Bot commented Sep 30, 2026

Code Coverage Summary

Uh oh!

github-actions Bot commented Sep 30, 2026

Code Coverage Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🔵 Needs a closer look

Uh oh!

github-actions Bot commented Sep 30, 2026

Code Coverage Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

github-actions Bot commented Sep 30, 2026

Code Coverage Summary

Uh oh!

Uh oh!

rdkdevpilot commented Sep 14, 2026 •

edited by tabbas651

Loading