[Home]

Summary:ASTERISK-27176: test_abstract_jb: frames leak
Reporter:Corey Farrell (coreyfarrell)Labels:
Date Opened:2017-08-03 23:03:22Date Closed:2021-09-10 12:17:06
Priority:MinorRegression?
Status:Closed/CompleteComponents:Core/Jitterbuffer Tests/General
Versions:GIT 15.0.0-beta1 Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) valgrind-AST_JB_ADAPTIVE_put_out_of_order.txt
( 1) valgrind-AST_JB_ADAPTIVE_put.txt
( 2) valgrind-AST_JB_FIXED_put_out_of_order.txt
( 3) valgrind-AST_JB_FIXED_put.txt
Description:test_abstract_jb leaks frames.  I'm unsure if the test or the jitter buffer implementation is the cause.

The following tests leak frames:
* AST_JB_ADAPTIVE_put
* AST_JB_ADAPTIVE_put_out_of_order
* AST_JB_FIXED_put
* AST_JB_FIXED_put_out_of_order
Comments:By: Asterisk Team (asteriskteam) 2017-08-03 23:03:22.921-0500

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].

By: Corey Farrell (coreyfarrell) 2017-08-03 23:08:10.366-0500

Attached are valgrind output for each test run separately (master).  The valgrind output was produced with frame caching disabled: I defined LOW_MEMORY for main/frame.c only.

By: Friendly Automation (friendly-automation) 2021-09-10 12:17:07.727-0500

Change 16451 merged by Friendly Automation:
test_abstract_jb.c: Fix put and put_out_of_order memory leaks.

[https://gerrit.asterisk.org/c/asterisk/+/16451|https://gerrit.asterisk.org/c/asterisk/+/16451]

By: Friendly Automation (friendly-automation) 2021-09-10 12:19:44.611-0500

Change 16453 merged by Friendly Automation:
test_abstract_jb.c: Fix put and put_out_of_order memory leaks.

[https://gerrit.asterisk.org/c/asterisk/+/16453|https://gerrit.asterisk.org/c/asterisk/+/16453]

By: Friendly Automation (friendly-automation) 2021-09-10 14:26:23.413-0500

Change 16419 merged by George Joseph:
test_abstract_jb.c: Fix put and put_out_of_order memory leaks.

[https://gerrit.asterisk.org/c/asterisk/+/16419|https://gerrit.asterisk.org/c/asterisk/+/16419]

By: Friendly Automation (friendly-automation) 2021-09-10 14:26:39.313-0500

Change 16452 merged by George Joseph:
test_abstract_jb.c: Fix put and put_out_of_order memory leaks.

[https://gerrit.asterisk.org/c/asterisk/+/16452|https://gerrit.asterisk.org/c/asterisk/+/16452]