From 765b7f81a1d157e809889c7224581db70a0ebe53 Mon Sep 17 00:00:00 2001
From: jvoisin
Date: Wed, 18 Oct 2017 00:31:18 +0200
Subject: Improve the strtol dance

---
 src/sp_config_keywords.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

(limited to 'src')

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) {
-- 
cgit v1.3