From 82fd985857ea6747579daf4f3c1bd905f9d00d61 Mon Sep 17 00:00:00 2001
From: Arkadiusz Bokowy <a.bokowy@samsung.com>
Date: Fri, 10 Jan 2025 09:50:11 +0100
Subject: [PATCH] Use shlex for joining command arguments

---
 scripts/tools/bouffalolab/generate_factory_data.py | 7 ++++---
 scripts/tools/zap_regen_all.py                     | 3 ++-
 third_party/tizen/tizen_dev_certificate.py         | 3 ++-
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/scripts/tools/bouffalolab/generate_factory_data.py b/scripts/tools/bouffalolab/generate_factory_data.py
index 28f362ba2d3b21..f6475ed07650d2 100755
--- a/scripts/tools/bouffalolab/generate_factory_data.py
+++ b/scripts/tools/bouffalolab/generate_factory_data.py
@@ -22,6 +22,7 @@
 import os
 import random
 import secrets
+import shlex
 import subprocess
 import sys
 from datetime import datetime, timedelta
@@ -185,7 +186,7 @@ def verify_certificates(chip_cert, paa_cert, pai_cert, dac_cert):
                        "--pai", pai_cert,
                        "--paa", paa_cert,
                        ]
-                log.info("Verify Certificate Chain: {}".format(" ".join(cmd)))
+                log.info("Verify Certificate Chain: {}".format(shlex.join(cmd)))
                 subprocess.run(cmd)
 
     def gen_dac_certificate(chip_cert, device_name, vendor_id, product_id, pai_cert, pai_key, dac_cert, dac_key, pai_issue_date, pai_expire_date):
@@ -214,7 +215,7 @@ def gen_valid_times(issue_date, expire_date):
                    "--valid-from", valid_from,
                    "--lifetime", str(lifetime),
                    ]
-            log.info("Generate DAC: {}".format(" ".join(cmd)))
+            log.info("Generate DAC: {}".format(shlex.join(cmd)))
             subprocess.run(cmd)
 
     def convert_pem_to_der(chip_cert, action, pem):
@@ -254,7 +255,7 @@ def gen_cd(chip_cert, dac_vendor_id, dac_product_id, vendor_id, product_id, cd_c
                     "--dac-origin-product-id", hex(dac_product_id),
                     ]
 
-        log.info("Generate CD: {}".format(" ".join(cmd)))
+        log.info("Generate CD: {}".format(shlex.join(cmd)))
         subprocess.run(cmd)
 
     pai_vendor_id, pai_product_id, pai_issue_date, pai_expire_date = parse_cert_file(pai_cert)
diff --git a/scripts/tools/zap_regen_all.py b/scripts/tools/zap_regen_all.py
index 0284e9eff9dd93..65fbab040ec58b 100755
--- a/scripts/tools/zap_regen_all.py
+++ b/scripts/tools/zap_regen_all.py
@@ -20,6 +20,7 @@
 import multiprocessing
 import os
 import os.path
+import shlex
 import shutil
 import subprocess
 import sys
@@ -209,7 +210,7 @@ def generate(self) -> TargetRunStats:
         """Runs a ZAP generate command on the configured zap/template/outputs.
         """
         cmd = self.build_cmd()
-        logging.info("Generating target: %s" % " ".join(cmd))
+        logging.info("Generating target: %s" % shlex.join(cmd))
 
         generate_start = time.time()
         subprocess.check_call(cmd)
diff --git a/third_party/tizen/tizen_dev_certificate.py b/third_party/tizen/tizen_dev_certificate.py
index 3d4fde970f432a..8f898949a70324 100755
--- a/third_party/tizen/tizen_dev_certificate.py
+++ b/third_party/tizen/tizen_dev_certificate.py
@@ -17,6 +17,7 @@
 import argparse
 import logging
 import os
+import shlex
 import subprocess
 import sys
 
@@ -29,7 +30,7 @@
 
 
 def run(cmd):
-    logging.debug("Run: %s", " ".join(cmd))
+    logging.debug("Run: %s", shlex.join(cmd))
     proc = subprocess.Popen(cmd, errors='replace',
                             stdout=subprocess.PIPE,
                             stderr=subprocess.PIPE)