Properly re.escape variant variable in find_used_variables_* (#5283)

Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu> Co-authored-by: Marcel Bargull <marcel.bargull@udo.edu>
conda · Apr 17, 2024 · 28b51fb · 28b51fb
1 parent 2d3270d
commit 28b51fb
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/conda_build/variants.py b/conda_build/variants.py
@@ -771,7 +771,7 @@ def find_used_variables_in_shell_script(variant, file_path):
         text = f.read()
     used_variables = set()
     for v in variant:
-        variant_regex = r"(^[^$]*?\$\{?\s*%s\s*[\s|\}])" % v
+        variant_regex = rf"(^[^$]*?\$\{{?\s*{re.escape(v)}\s*[\s|\}}])"
         if re.search(variant_regex, text, flags=re.MULTILINE | re.DOTALL):
             used_variables.add(v)
     return used_variables
@@ -782,7 +782,7 @@ def find_used_variables_in_batch_script(variant, file_path):
         text = f.read()
     used_variables = set()
     for v in variant:
-        variant_regex = r"\%" + v + r"\%"
+        variant_regex = rf"\%{re.escape(v)}\%"
         if re.search(variant_regex, text, flags=re.MULTILINE | re.DOTALL):
             used_variables.add(v)
     return used_variables