switch back to len(min of parts); avoids OOB array access

bazelbuild · Nov 25, 2024 · 4720a07 · 4720a07
1 parent 2b362aa
commit 4720a07
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/python/private/py_executable_bazel.bzl b/python/private/py_executable_bazel.bzl
@@ -360,7 +360,8 @@ def relative_path(from_, to):
     to_parts = to.split("/")
 
     # Strip common leading parts from both paths
-    for _ in range(len(to)):
+    n = min(len(from_parts), len(to_parts))
+    for _ in range(n):
         if from_parts[0] == to_parts[0]:
             from_parts.pop(0)
             to_parts.pop(0)