src: use starts_with in fs_permission.cc

IlyasShabi · aduh95 · commit 2a5543b78ee1 · 2025-01-02T15:28:48.000+01:00
PR-URL: #55811 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jason Zhang <xzha4350@gmail.com> Reviewed-By: Deokjin Kim <deokjin81.kim@gmail.com>
diff --git a/src/permission/fs_permission.cc b/src/permission/fs_permission.cc
@@ -58,16 +58,16 @@ bool is_tree_granted(
   std::string resolved_param = node::PathResolve(env, {param});
 #ifdef _WIN32
   // Remove leading "\\?\" from UNC path
-  if (resolved_param.substr(0, 4) == "\\\\?\\") {
+  if (resolved_param.starts_with("\\\\?\\")) {
     resolved_param.erase(0, 4);
   }
 
   // Remove leading "UNC\" from UNC path
-  if (resolved_param.substr(0, 4) == "UNC\\") {
+  if (resolved_param.starts_with("UNC\\")) {
     resolved_param.erase(0, 4);
   }
   // Remove leading "//" from UNC path
-  if (resolved_param.substr(0, 2) == "//") {
+  if (resolved_param.starts_with("//")) {
     resolved_param.erase(0, 2);
   }
 #endif

Original file line number	Diff line number	Diff line change
`@@ -58,16 +58,16 @@ bool is_tree_granted(`
`58`	`58`	`std::string resolved_param = node::PathResolve(env, {param});`
`59`	`59`	`#ifdef _WIN32`
`60`	`60`	`// Remove leading "\\?\" from UNC path`
`61`		`- if (resolved_param.substr(0, 4) == "\\\\?\\") {`
	`61`	`+ if (resolved_param.starts_with("\\\\?\\")) {`
`62`	`62`	`resolved_param.erase(0, 4);`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`// Remove leading "UNC\" from UNC path`
`66`		`- if (resolved_param.substr(0, 4) == "UNC\\") {`
	`66`	`+ if (resolved_param.starts_with("UNC\\")) {`
`67`	`67`	`resolved_param.erase(0, 4);`
`68`	`68`	`}`
`69`	`69`	`// Remove leading "//" from UNC path`
`70`		`- if (resolved_param.substr(0, 2) == "//") {`
	`70`	`+ if (resolved_param.starts_with("//")) {`
`71`	`71`	`resolved_param.erase(0, 2);`
`72`	`72`	`}`
`73`	`73`	`#endif`