Reviewed-by: Nicholas Pratte <npra...@iol.unh.edu>
On Fri, Mar 14, 2025 at 9:19 AM Luca Vizzarro <luca.vizza...@arm.com> wrote:
>
> When making any command a privileged one in a LinuxSession, there
> currently is no consideration whether this command already includes single
> quotes. Therefore escape the existing single quotes before making the
> command.
>
> Signed-off-by: Luca Vizzarro <luca.vizza...@arm.com>
> Reviewed-by: Paul Szczepanek <paul.szczepa...@arm.com>
> ---
> dts/framework/testbed_model/linux_session.py | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/dts/framework/testbed_model/linux_session.py
> b/dts/framework/testbed_model/linux_session.py
> index 7c2b110c99..6c6a4b608d 100644
> --- a/dts/framework/testbed_model/linux_session.py
> +++ b/dts/framework/testbed_model/linux_session.py
> @@ -67,6 +67,7 @@ class LinuxSession(PosixSession):
>
> @staticmethod
> def _get_privileged_command(command: str) -> str:
> + command = command.replace(r"'", r"\'")
> return f"sudo -- sh -c '{command}'"
>
> def get_remote_cpus(self) -> list[LogicalCore]:
> --
> 2.43.0
>
