fix issue 716 (#717)

Author: lemire

include/ada/url-inl.h

@@ -215,10 +215,15 @@ inline void url::copy_scheme(const ada::url &u) {
 ada_really_inline size_t url::parse_port(std::string_view view,
                                          bool check_trailing_content) noexcept {
   ada_log("parse_port('", view, "') ", view.size());
+  if (!view.empty() && view[0] == '-') {
+    ada_log("parse_port: view[0] == '0' && view.size() > 1");
+    is_valid = false;
+    return 0;
+  }
   uint16_t parsed_port{};
   auto r = std::from_chars(view.data(), view.data() + view.size(), parsed_port);
   if (r.ec == std::errc::result_out_of_range) {
-    ada_log("parse_port: std::errc::result_out_of_range");
+    ada_log("parse_port: r.ec == std::errc::result_out_of_range");
     is_valid = false;
     return 0;
   }

include/ada/url_aggregator-inl.h

@@ -853,10 +853,15 @@ inline bool url_aggregator::has_port() const noexcept {
 ada_really_inline size_t url_aggregator::parse_port(
     std::string_view view, bool check_trailing_content) noexcept {
   ada_log("url_aggregator::parse_port('", view, "') ", view.size());
+  if (!view.empty() && view[0] == '-') {
+    ada_log("parse_port: view[0] == '0' && view.size() > 1");
+    is_valid = false;
+    return 0;
+  }
   uint16_t parsed_port{};
   auto r = std::from_chars(view.data(), view.data() + view.size(), parsed_port);
   if (r.ec == std::errc::result_out_of_range) {
-    ada_log("parse_port: std::errc::result_out_of_range");
+    ada_log("parse_port: r.ec == std::errc::result_out_of_range");
     is_valid = false;
     return 0;
   }

tests/basic_tests.cpp

@@ -440,4 +440,10 @@ TYPED_TEST(basic_tests, path_setter_bug) {
   ASSERT_TRUE(base_url->set_pathname("//.."));
   ASSERT_TRUE(base_url->validate());
   SUCCEED();
+}
+
+TYPED_TEST(basic_tests, negativeport) {
+  auto url = ada::parse<TypeParam>("https://www.google.com");
+  ASSERT_FALSE(url->set_port("-1"));
+  SUCCEED();
 }