[Asterisk-code-review] res pjsip: qualify/auth test failed with ref leaks when usin... (testsuite[master])

Jenkins2 asteriskteam at digium.com
Wed Nov 1 09:40:26 CDT 2017


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/6943 )

Change subject: res_pjsip: qualify/auth test failed with ref leaks when using REF_DEBUG
......................................................................

res_pjsip: qualify/auth test failed with ref leaks when using REF_DEBUG

When REF_DEBUG was enabled the pjsip/qualify/auth would fail due to the
req_wrapper reference being leaked. This happened because a second QUALIFY
was being sent out while the test was ending (Asterisk was being shutdown).
So the send request callback was never called, due to Asterisk disappearing,
thus not releasing the memory.

For now the solution is to just increase the qualify frequency so only a
single qualify is sent for the test.

ASTERISK-27358 #close

Change-Id: If597268978ea79852e9e339a2a56a8ac7d2f63dd
---
M tests/channels/pjsip/qualify/auth/configs/ast1/pjsip.conf
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Corey Farrell: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/tests/channels/pjsip/qualify/auth/configs/ast1/pjsip.conf b/tests/channels/pjsip/qualify/auth/configs/ast1/pjsip.conf
index 9340824..1b55a94 100644
--- a/tests/channels/pjsip/qualify/auth/configs/ast1/pjsip.conf
+++ b/tests/channels/pjsip/qualify/auth/configs/ast1/pjsip.conf
@@ -10,4 +10,4 @@
 [sipp]
 type=aor
 contact=sip:127.0.0.1:5061
-qualify_frequency=1
+qualify_frequency=7

-- 
To view, visit https://gerrit.asterisk.org/6943
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If597268978ea79852e9e339a2a56a8ac7d2f63dd
Gerrit-Change-Number: 6943
Gerrit-PatchSet: 1
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171101/f68c9916/attachment.html>


More information about the asterisk-code-review mailing list