[asterisk-commits] twilson: branch certified-1.8.11 r370472 - /certified/branches/1.8.11/main/pbx.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Jul 24 22:37:12 CDT 2012


Author: twilson
Date: Tue Jul 24 22:37:08 2012
New Revision: 370472

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=370472
Log:
Revert a change that broke compilation

1) There is no such function as ast_ref()
2) The patch was originally credited as the one uploaded by Guenther
   Kelleter (license 6372) via issue AST-921, but the patch committed
   was not the patch referenced on the issue.
3) Guenther Kelleter's patch was actually correct. It moved the
   ast_free above the presencechange_cleanup label. I am not
   committing his change as it is not technically necesary--calling
   ast_free(NULL) is perfectly safe and I worry that moving the
   ast_free outside of the label could lead to future bugs if
   someone ever adds another failure conditional and expects
   'goto presencechange_cleanup;' to clean up after everything.

Modified:
    certified/branches/1.8.11/main/pbx.c

Modified: certified/branches/1.8.11/main/pbx.c
URL: http://svnview.digium.com/svn/asterisk/certified/branches/1.8.11/main/pbx.c?view=diff&rev=370472&r1=370471&r2=370472
==============================================================================
--- certified/branches/1.8.11/main/pbx.c (original)
+++ certified/branches/1.8.11/main/pbx.c Tue Jul 24 22:37:08 2012
@@ -4644,7 +4644,7 @@
 	res = 0;
 
 presencechange_cleanup:
-	ast_ref(hint_app, -1);
+	ast_free(hint_app);
 	ao2_ref(pc, -1);
 
 	return res;




More information about the asterisk-commits mailing list