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

[Kevin Harwell]

Kevin Harwell has uploaded this change for review. ( 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(-)



  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/43/6943/1

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: newchange
Gerrit-Change-Id: If597268978ea79852e9e339a2a56a8ac7d2f63dd
Gerrit-Change-Number: 6943
Gerrit-PatchSet: 1
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171031/48f0f73e/attachment.html>