Improve the strtol dance

1 files changed, 8 insertions, 4 deletions

diff --git a/src/sp_config_keywords.c b/src/sp_config_keywords.c
index cbdd579..b03c28e 100644
--- a/src/sp_config_keywords.c
+++ b/src/sp_config_keywords.c
@@ -239,13 +239,17 @@ int parse_disabled_functions(char *line) {
   if (pos) {
     errno = 0;
     char *endptr;
-    df->pos = strtol(pos, &endptr, 10) > 128 ? 128 : strtol(pos, NULL, 10);
+    df->pos = strtol(pos, &endptr, 10);
     if (errno != 0 || endptr == pos) {
-      sp_log_err("config",
-                 "Failed to parse arg '%s' of `pos` on line %zu.",
-                 pos, sp_line_no);
+      sp_log_err("config", "Failed to parse arg '%s' of `pos` on line %zu.",
+                    pos, sp_line_no);
       return -1;
     }
+
+    // We'll never have a function with more than 128 params
+    if (df->pos > 128) {
+      df->pos = 128;
+    }
   }
 
   if (df->function) {