[asterisk-dev] Change in testsuite[master]: res_phoneprov_pjsip: Disconnect HTTP before stopping Asterisk.

Corey Farrell (Code Review) asteriskteam at digium.com
Wed Apr 8 21:35:51 CDT 2015


Corey Farrell has uploaded a new change for review.

  https://gerrit.asterisk.org/35

Change subject: res_phoneprov_pjsip: Disconnect HTTP before stopping Asterisk.
......................................................................

res_phoneprov_pjsip: Disconnect HTTP before stopping Asterisk.

Leaving an HTTP connection open causes Asterisk to report a reference
leak.  This change ensures the connection is closed before stopping
Asterisk.

Change-Id: I5553af9cbfdc1d68110036eaadcfe5db3570928f
---
M tests/phoneprov/res_phoneprov_pjsip/run-test
1 file changed, 5 insertions(+), 0 deletions(-)


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

diff --git a/tests/phoneprov/res_phoneprov_pjsip/run-test b/tests/phoneprov/res_phoneprov_pjsip/run-test
index 9288929..cf52311 100755
--- a/tests/phoneprov/res_phoneprov_pjsip/run-test
+++ b/tests/phoneprov/res_phoneprov_pjsip/run-test
@@ -65,6 +65,11 @@
 
         self.stop_reactor()
 
+    def stop_reactor(self):
+        self.con.close()
+        super(ResPhoneprovTest, self).stop_reactor()
+        
+
 def main():
     test = ResPhoneprovTest()
     reactor.run()

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5553af9cbfdc1d68110036eaadcfe5db3570928f
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Corey Farrell <git at cfware.com>



More information about the asterisk-dev mailing list