Skip to content

RDKB-66073 : Observed OVSM memory leak during VIF_Config table update#1272

Merged
gsathish86 merged 2 commits into
rdkcentral:developfrom
lavanyachimmiri15:bug/RDKB-66073
Jul 16, 2026
Merged

RDKB-66073 : Observed OVSM memory leak during VIF_Config table update#1272
gsathish86 merged 2 commits into
rdkcentral:developfrom
lavanyachimmiri15:bug/RDKB-66073

Conversation

@lavanyachimmiri15

Copy link
Copy Markdown
Contributor

RDKB-66073 : Observed OVSM memory leak during VIF_Config table update

Reason for change: Observed OVSM memory leak during VIF_Config table update
Test Procedure: Update the Wifi_VIF_Config table manually and ensure that no memory spike is observed during or after the update. Risks: Medium
Priority: P1
Signed-off-by:lavanya_chimmirivenkata@comcast.com

Copilot AI review requested due to automatic review settings July 15, 2026 12:56
@lavanyachimmiri15
lavanyachimmiri15 requested a review from a team as a code owner July 15, 2026 12:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes an OVSM memory leak observed during Wifi_VIF_Config table updates by ensuring encoded subdoc raw buffers are released when the incoming update results in “no config change”, preventing repeated allocations from accumulating.

Changes:

  • Free webconfig_ovsdb_data.u.encoded.raw on the early-return “no change” path in webconfig_ovsdb_encode() to avoid leaking the newly encoded JSON blob.

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

Comment thread source/webconfig/wifi_ovsdb_translator.c Outdated
Copilot AI review requested due to automatic review settings July 16, 2026 04:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread source/webconfig/wifi_ovsdb_translator.c
Copilot AI review requested due to automatic review settings July 16, 2026 07:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread source/webconfig/wifi_ovsdb_translator.c
Reason for change: Observed OVSM memory leak during VIF_Config table update
Test Procedure: Update the Wifi_VIF_Config table manually and ensure that no memory spike is observed during or after the update.
Risks: Medium
Priority: P1
Signed-off-by:lavanya_chimmirivenkata@comcast.com
Copilot AI review requested due to automatic review settings July 16, 2026 08:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

@gsathish86
gsathish86 merged commit fe7f981 into rdkcentral:develop Jul 16, 2026
8 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants