Remove unnecessary and dangerous path string parsing

Tom-Newton · Oct 18, 2023 · 8e9a985 · 8e9a985
1 parent 835e6ab
commit 8e9a985
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/cpp/src/arrow/filesystem/azurefs.cc b/cpp/src/arrow/filesystem/azurefs.cc
@@ -78,9 +78,7 @@ struct AzurePath {
           "Expected an Azure object path of the form 'container/path...', got a URI: '",
           s, "'");
     }
-    auto src = internal::RemoveTrailingSlash(s);
-    auto input_path = std::string(src.data());
-    src = internal::RemoveLeadingSlash(src);
+    const auto src = internal::RemoveTrailingSlash(s);
     auto first_sep = src.find_first_of(internal::kSep);
     if (first_sep == 0) {
       return Status::Invalid("Path cannot start with a separator ('", s, "')");