revert my pass through the tree to remove checks of the result of ast_strdupa
[asterisk/asterisk.git] / funcs / func_logic.c
old mode 100755 (executable)
new mode 100644 (file)
index a195eca..d1d96ab
@@ -19,6 +19,7 @@
  * 
  * \brief Conditional logic dialplan functions
  * 
+ * \author Anthony Minessale II
  */
 
 #include <stdlib.h>
@@ -64,7 +65,7 @@ static char *builtin_function_iftime(struct ast_channel *chan, char *cmd, char *
        iftrue = strsep(&data, ":");
        iffalse = data;
 
-       if (!expr || ast_strlen_zero(expr) || !(iftrue || iffalse)) {
+       if (ast_strlen_zero(expr) || !(iftrue || iffalse)) {
                ast_log(LOG_WARNING, "Syntax IFTIME(<timespec>?[<true>][:<false>])\n");
                return NULL;
        }
@@ -104,7 +105,7 @@ static char *builtin_function_if(struct ast_channel *chan, char *cmd, char *data
        iftrue = strsep(&data, ":");
        iffalse = data;
 
-       if (!expr || ast_strlen_zero(expr) || !(iftrue || iffalse)) {
+       if (ast_strlen_zero(expr) || !(iftrue || iffalse)) {
                ast_log(LOG_WARNING, "Syntax IF(<expr>?[<true>][:<false>])\n");
                return NULL;
        }
@@ -136,7 +137,7 @@ static char *builtin_function_set(struct ast_channel *chan, char *cmd, char *dat
        varname = strsep(&data, "=");
        val = data;
 
-       if (!varname || ast_strlen_zero(varname) || !val) {
+       if (ast_strlen_zero(varname) || !val) {
                ast_log(LOG_WARNING, "Syntax SET(<varname>=[<value>])\n");
                return NULL;
        }