[PATCH 2/2] binman: nxp_imx8mcst: disable configparser interpolation

[Quentin Schulz]

Hi Julian,

On 5/4/26 6:48 PM, julian.haller at bbl.ms.philips.com wrote:
> [You don't often get email from julian.haller at bbl.ms.philips.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> 
> From: Julian Haller <julian.haller at philips.com>
> 
> 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 <julian.haller at philips.com>
> ---
>   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
>