Fix glob matching algo

There was one issue when you were using only `?`.
I've added a circuit break that fixes that issue and also improve the perf of the algo in the case of patterns with no `*` and shorter than the length of the input string.

Author: pierotibou

ext/ddshared.c

@@ -42,10 +42,19 @@ bool dd_glob_rule_matches(zval *pattern, zend_string* value) {
     char *s = ZSTR_VAL(value);
 
     int wildcards = 0;
+    int patternLength = 0;
+    int stringLength = ZSTR_LEN(value);
     while (*p) {
         if (*(p++) == '*') {
             ++wildcards;
         }
+        patternLength++;
+    }
+
+    // If there are no wildcards, no need to go through the whole string if pattern is shorter than the input string
+    // Indeed wildcards (ie '*') can replace multiple characters while '?' canonly replace one
+    if (wildcards == 0 && patternLength < stringLength) {
+        return false;
     }
 
     p = Z_STRVAL_P(pattern);

tests/ext/priority_sampling/021-rule-name-as-glob-with-question-mark.phpt

@@ -12,6 +12,8 @@ $tests = [
     ["**fooname", "fooname", true],
     ["*", "fooname", true],
     ["???????", "fooname", true],
+    ["??????", "fooname", false],
+    ["??", "fooname", false],
     ["*?", "fooname", true],
     ["?*", "fooname", true],
     ["f*o*e", "fooname", true],
@@ -61,6 +63,10 @@ As expected, * matches fooname (name)
 As expected, * matches fooname (service)
 As expected, ??????? matches fooname (name)
 As expected, ??????? matches fooname (service)
+As expected, ?????? doesn't match fooname (name)
+As expected, ?????? doesn't match fooname (service)
+As expected, ?? doesn't match fooname (name)
+As expected, ?? doesn't match fooname (service)
 As expected, *? matches fooname (name)
 As expected, *? matches fooname (service)
 As expected, ?* matches fooname (name)