If the addr pointer is NULL, better to check first before using it.

2013-06-12 21:23:52 -07:00 · 2013-06-12 21:23:52 -07:00 · fef983f614
parent 7360c1e898
commit fef983f614
1 changed files with 2 additions and 3 deletions
--- a/src/mod/applications/mod_expr/exprval.c
+++ b/src/mod/applications/mod_expr/exprval.c
@ -213,12 +213,11 @@ int exprValListGetAddress(exprValList * vlist, char *name, EXPRTYPE ** addr)
 	exprVal *cur;
 	int result;

-	/* Not found yet */
-	*addr = NULL;
-
 	if (vlist == NULL || addr == NULL)
 		return EXPR_ERROR_NULLPOINTER;

+	/* Not found yet */
+	*addr = NULL;

 	if (name == NULL || name[0] == '\0')
 		return EXPR_ERROR_NOTFOUND;