[asterisk-commits] tilghman: branch tilghman/str_substitution r186053 - /team/tilghman/str_subst...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Apr 2 10:28:04 CDT 2009


Author: tilghman
Date: Thu Apr  2 10:27:58 2009
New Revision: 186053

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=186053
Log:
Remove redundant checks, from Dimas' review.
http://reviewboard.digium.com/r/174/

Modified:
    team/tilghman/str_substitution/apps/app_voicemail.c

Modified: team/tilghman/str_substitution/apps/app_voicemail.c
URL: http://svn.digium.com/svn-view/asterisk/team/tilghman/str_substitution/apps/app_voicemail.c?view=diff&rev=186053&r1=186052&r2=186053
==============================================================================
--- team/tilghman/str_substitution/apps/app_voicemail.c (original)
+++ team/tilghman/str_substitution/apps/app_voicemail.c Thu Apr  2 10:27:58 2009
@@ -4019,12 +4019,8 @@
 	char filename[256];
 
 	if (!str1 || !str2) {
-		if (str1) {
-			ast_free(str1);
-		}
-		if (str2) {
-			ast_free(str2);
-		}
+		ast_free(str1);
+		ast_free(str2);
 		return;
 	}
 #ifdef IMAP_STORAGE
@@ -4320,12 +4316,8 @@
 	struct ast_str *str1 = ast_str_create(16), *str2 = ast_str_create(16);
 
 	if (!str1 || !str2) {
-		if (str1) {
-			ast_free(str1);
-		}
-		if (str2) {
-			ast_free(str2);
-		}
+		ast_free(str1);
+		ast_free(str2);
 		return -1;
 	}
 




More information about the asterisk-commits mailing list