fix URIParser bug

sogou · Dec 3, 2021 · a13d5d5 · a13d5d5
1 parent 802c224
commit a13d5d5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/util/URIParser.cc b/src/util/URIParser.cc
@@ -327,6 +327,9 @@ int URIParser::parse(const char *str, ParsedURI& uri)
  break;
 
  case URI_PATH:
+ if (skip_path)
+ continue;
+
  start_idx[URI_PATH] = i;
  break;
 

diff --git a/test/uriparser_unittest.cc b/test/uriparser_unittest.cc
@@ -207,5 +207,9 @@ TEST(uriparser_unittest, parse)
  EXPECT_EQ(uri.path, nullptr);
  EXPECT_EQ(uri.query, nullptr);
  EXPECT_EQ(uri.fragment, nullptr);
+
+ EXPECT_EQ(URIParser::parse("http://www.sogou.com?aaa/=bbb", uri), 0);
+ EXPECT_EQ(uri.path, nullptr);
+ EXPECT_EQ(strcmp(uri.query, "aaa/=bbb"), 0);
 }