Remove the GLOB_BRACE flag

When calling glob() to get the configuration file location, the
GLOB_BRACE flag is used. This flag enables the parsing of '{a,b}', so
that '/path/{a,b}' represents '/path/a' and '/path/b'. Looking at
Snuffleupagus' documentation I saw nowhere this kind of path was
supported. Instead the documentation shows that glob() is used for paths
with wildcards.

The use of GLOB_BRACE is problematic as it is glibc specific and is not
supported by some other C libraries, such as Musl. Snuffleupagus cannot
be used in Apline for this reason.

Since the documentation does not give a valid usage of GLOB_BRACE this
patch removes it, which fixes the non-glibc support.

Signed-off-by: Antoine Tenart <antoine.tenart@ack.tf>

1 files changed, 1 insertions, 1 deletions

diff --git a/src/snuffleupagus.c b/src/snuffleupagus.c
index 9f92397..ca23bac 100644
--- a/src/snuffleupagus.c
+++ b/src/snuffleupagus.c
@@ -224,7 +224,7 @@ static PHP_INI_MH(OnUpdateConfiguration) {
   char *rest = new_value->val;
 
   while ((config_file = strtok_r(rest, ",", &rest))) {
-    int ret = glob(config_file, GLOB_BRACE | GLOB_NOCHECK, NULL, &globbuf);
+    int ret = glob(config_file, GLOB_NOCHECK, NULL, &globbuf);
 
     if (ret != 0) {
       SNUFFLEUPAGUS_G(is_config_valid) = false;