Refactor: libcrmcommon: Drop pcmk__valid_script()

Unused since 73a5d63. Signed-off-by: Reid Wahl <nrwahl@protonmail.com>
ClusterLabs · Mar 26, 2024 · b1a5381 · b1a5381
1 parent 057cd28
commit b1a5381
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 38 deletions.
diff --git a/include/crm/common/options_internal.h b/include/crm/common/options_internal.h
@@ -117,7 +117,6 @@ bool pcmk__valid_int(const char *value);
 bool pcmk__valid_positive_int(const char *value);
 bool pcmk__valid_no_quorum_policy(const char *value);
 bool pcmk__valid_percentage(const char *value);
-bool pcmk__valid_script(const char *value);
 bool pcmk__valid_placement_strategy(const char *value);
 
 // from watchdog.c

diff --git a/lib/common/options.c b/lib/common/options.c
@@ -1313,43 +1313,6 @@ pcmk__valid_percentage(const char *value)
     return ((end == NULL) || (end[0] == '%')) && (number >= 0);
 }
 
-/*!
- * \internal
- * \brief Check whether a string represents a valid script
- *
- * Valid values include \c /dev/null and paths of executable regular files
- *
- * \param[in] value  String to validate
- *
- * \return \c true if \p value is a valid script, or \c false otherwise
- */
-bool
-pcmk__valid_script(const char *value)
-{
-    struct stat st;
-
-    if (pcmk__str_eq(value, "/dev/null", pcmk__str_none)) {
-        return true;
-    }
-
-    if (stat(value, &st) != 0) {
-        crm_err("Script %s does not exist", value);
-        return false;
-    }
-
-    if (S_ISREG(st.st_mode) == 0) {
-        crm_err("Script %s is not a regular file", value);
-        return false;
-    }
-
-    if ((st.st_mode & (S_IXUSR | S_IXGRP)) == 0) {
-        crm_err("Script %s is not executable", value);
-        return false;
-    }
-
-    return true;
-}
-
 /*!
  * \internal
  * \brief Check whether a string represents a valid placement strategy