fix: parse_url with fragment (apache#166)

zhouyuan · Sep 14, 2022 · e9872bb · e9872bb
1 parent d389aab
commit e9872bb
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cpp/src/gandiva/parse_url_holder.h b/cpp/src/gandiva/parse_url_holder.h
@@ -73,7 +73,7 @@ namespace gandiva {
       }
       // check invalid charactor in fragment
       if (fragment_start_idx != -1) {
-        for (int32_t idx = fragment_start_idx; idx < url_len; idx++) {
+        for (int32_t idx = fragment_start_idx + 1; idx < url_len; idx++) {
           // consist with vanilla spark
           if (url[idx] == ' ' || url[idx] == '{' || url[idx] == '}' || url[idx] == '#') {
             return nullptr;
@@ -199,7 +199,7 @@ namespace gandiva {
       }
       // check invalid charactor in fragment
       if (fragment_start_idx != -1) {
-        for (int32_t idx = fragment_start_idx; idx < url_len; idx++) {
+        for (int32_t idx = fragment_start_idx + 1; idx < url_len; idx++) {
           // consist with vanilla spark
           if (url[idx] == ' ' || url[idx] == '{' || url[idx] == '}' || url[idx] == '#') {
             return nullptr;