Hi Julian,
On 5/4/26 6:48 PM, [email protected] wrote:
[You don't often get email from [email protected]. Learn why
this is important at https://aka.ms/LearnAboutSenderIdentification ]
From: Julian Haller <[email protected]>
ConfigParser uses %-based interpolation by default, which causes
PKCS#11 URIs containing percent-encoded characters (e.g. %20) to
fail with "invalid interpolation syntax". Disable interpolation
since the CST config format does not use this feature.
Should we add a test in tools/binman/ftest.py for that corner case so we
don't regress?
The change makes sense to me though I don't know how/where we're
supposed to pass a PKCS#11 URI to CST, so I'll let Simon/Marek discuss that.
Cheers,
Quentin
Signed-off-by: Julian Haller <[email protected]>
---
tools/binman/etype/nxp_imx8mcst.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/binman/etype/nxp_imx8mcst.py
b/tools/binman/etype/nxp_imx8mcst.py
index dd9f226b751..4fa26de51a0 100644
--- a/tools/binman/etype/nxp_imx8mcst.py
+++ b/tools/binman/etype/nxp_imx8mcst.py
@@ -137,7 +137,7 @@ class Entry_nxp_imx8mcst(Entry_mkimage):
# Generate CST configuration file used to sign payload
cfg_fname = tools.get_output_filename(f'nxp.csf-config-txt.{uniq}')
- config = configparser.ConfigParser()
+ config = configparser.ConfigParser(interpolation=None)
# Do not make key names lowercase
config.optionxform = str
# Load configuration template and modify keys of interest
--
2.43.0
