From d934db81cdcd64086c8867bb422bfa7e0d18bb38 Mon Sep 17 00:00:00 2001
From: jvoisin
Date: Sun, 9 May 2021 22:55:36 +0200
Subject: strtok/strtok_r is a thing from the past, don't use it.

---
 src/snuffleupagus.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/snuffleupagus.c b/src/snuffleupagus.c
index f360a0c..f0737b4 100644
--- a/src/snuffleupagus.c
+++ b/src/snuffleupagus.c
@@ -250,14 +250,15 @@ static PHP_INI_MH(OnUpdateConfiguration) {
     return FAILURE;
   }
 
-  glob_t globbuf;
-  char *config_file;
-  char *rest = new_value->val;
+  char *str = new_value->val;
 
-  while ((config_file = strtok_r(rest, ",", &rest))) {
-    int ret = glob(config_file, GLOB_NOCHECK, NULL, &globbuf);
+  while (1) {
+    // We don't care about overwriting new_value->val
+    char *config_file = strsep(&str, ",");
+    if (config_file == NULL) break;
 
-    if (ret != 0) {
+    glob_t globbuf;
+    if (0 != glob(config_file, GLOB_NOCHECK, NULL, &globbuf)) {
       SNUFFLEUPAGUS_G(is_config_valid) = SP_CONFIG_INVALID;
       globfree(&globbuf);
       return FAILURE;
-- 
cgit v1.3