Proper return code check

FreeRADIUS · Apr 20, 2014 · eefa43e · eefa43e
1 parent bd793f9
commit eefa43e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/modcall.c b/src/main/modcall.c
@@ -652,7 +652,8 @@ static bool modcall_recurse(REQUEST *request, rlm_components_t component, int de
 			goto calculate_result;
 		}
 
-		if (radius_vpt_get_vp(&vp, request, g->vpt) <= 0) {	/* nothing to loop over */
+		if ((radius_vpt_get_vp(&vp, request, g->vpt) < 0) || /* nothing to loop over */
+		    !vp) {
 			MOD_LOG_OPEN_BRACE("foreach");
 			result = RLM_MODULE_NOOP;
 			MOD_LOG_CLOSE_BRACE();