Merge pull request #5729 from btriller/cf-agent-files

cf-agent/verify_files: Drop duplicate check on present select attribute

Author: olehermanse

cf-agent/verify_files.c

@@ -130,7 +130,7 @@ static bool FileSanityChecks(char *path, const Attributes *a, const Promise *pp)
 
     if ((a->haveselect) && (!a->select.result))
     {
-        Log(LOG_LEVEL_ERR, "File select constraint body promised no result (check body definition)");
+        Log(LOG_LEVEL_ERR, "File selection failed: check file_select body definition; file_result attribute is not defined.");
         PromiseRef(LOG_LEVEL_ERR, pp);
         return false;
     }
@@ -165,13 +165,6 @@ static bool FileSanityChecks(char *path, const Attributes *a, const Promise *pp)
         return false;
     }
 
-    if ((a->haveselect) && (a->select.result == NULL))
-    {
-        Log(LOG_LEVEL_ERR, "Missing file_result attribute in file_select body");
-        PromiseRef(LOG_LEVEL_ERR, pp);
-        return false;
-    }
-
     if ((a->havedepthsearch) && (a->change.report_diffs))
     {
         Log(LOG_LEVEL_ERR, "Difference reporting is not allowed during a depth_search");