fixes a few redundant conditions

(issue #15269)


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

main/ast_expr2.c

@@ -2415,7 +2415,6 @@ static void
 free_value (struct val *vp)
 {	
 	if (vp==NULL) {
-		free(vp);
 		return;
 	}
 	if (vp->type == AST_EXPR_string || vp->type == AST_EXPR_numeric_string)

main/ast_expr2.fl

@@ -243,7 +243,7 @@ int ast_yyerror(const char *, YYLTYPE *, struct parse_io *); /* likewise */
 
 void ast_yyfree(void *ptr, yyscan_t yyscanner)
 {
-      if (ptr) /* the normal generated yyfree func just frees its first arg;
+    /* the normal generated yyfree func just frees its first arg;
      this get complaints on some systems, as sometimes this
      arg is a nil ptr! It's usually not fatal, but is irritating! */
 	free( (char *) ptr );
@@ -287,7 +287,6 @@ int ast_expr(char *expr, char *buf, int length, struct ast_channel *chan)
 			else
 				buf[0] = 0;
 			return_value = strlen(buf);
-			if (io.val->u.s)
 			free(io.val->u.s);
 		}
 		free(io.val);