]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Fix memory leak
authorKevin Harwell <kharwell@digium.com>
Fri, 30 Aug 2013 17:00:26 +0000 (17:00 +0000)
committerKevin Harwell <kharwell@digium.com>
Fri, 30 Aug 2013 17:00:26 +0000 (17:00 +0000)
Fixed a features.c test that leaked a reference to a parked call.  This caused
chancount to never reach 0, so graceful shutdown stops.  Also added an
unregister test.

(closes issue ASTERISK-22413)
Reported by: Corey Farrell
Patches:
     features-TEST_FRAMEWORK.patch uploaded by coreyfarrell (license 5909)
........

Merged revisions 398021 from http://svn.asterisk.org/svn/asterisk/branches/1.8

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@398022 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/features.c

index 3e824752f9e22a8102537c753ba8a10b529e0deb..0a3a24bd64fcd902d2f206c8801223534dc7f75b 100644 (file)
@@ -8850,6 +8850,7 @@ AST_TEST_DEFINE(features_test)
                }
                res = -1;
        }
+       parked_chan = ast_channel_unref(parked_chan);
 
 
 exit_features_test:
@@ -9035,6 +9036,9 @@ static void features_shutdown(void)
        ast_unregister_application(parkcall);
        ast_unregister_application(parkedcall);
        ast_unregister_application(app_bridge);
+#if defined(TEST_FRAMEWORK)
+       AST_TEST_UNREGISTER(features_test);
+#endif /* defined(TEST_FRAMEWORK) */
 
        pthread_cancel(parking_thread);
        pthread_kill(parking_thread, SIGURG);