docs: kernel_feat.py: fix potential command injection

commit c48a7c44a1 upstream. The kernel-feat directive passes its argument straight to the shell. This is unfortunate and unnecessary. Let's always use paths relative to $srctree/Documentation/ and use subprocess.check_call() instead of subprocess.Popen(shell=True). This also makes the code shorter. This is analogous to commit 3231dd5862 ("docs: kernel_abi.py: fix command injection") where we did exactly the same thing for kernel_abi.py, somehow I completely missed this one. Link: https://fosstodon.org/@jani/111676532203641247 Reported-by: Jani Nikula <jani.nikula@intel.com> Signed-off-by: Vegard Nossum <vegard.nossum@oracle.com> Cc: stable@vger.kernel.org Signed-off-by: Jonathan Corbet <corbet@lwn.net> Link: https://lore.kernel.org/r/20240110174758.3680506-1-vegard.nossum@oracle.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2026-10-09 18:15:49 +00:00 · 2024-01-10 18:47:58 +01:00 · 2024-01-10 18:47:58 +01:00 · f7f4ff54c4
commit f7f4ff54c4
parent 2f2eb48bbe
22 changed files with 32 additions and 65 deletions
--- a/Documentation/admin-guide/features.rst
+++ b/Documentation/admin-guide/features.rst
--- a/Documentation/arch/arc/features.rst
+++ b/Documentation/arch/arc/features.rst
--- a/Documentation/arch/arm/features.rst
+++ b/Documentation/arch/arm/features.rst
--- a/Documentation/arch/arm64/features.rst
+++ b/Documentation/arch/arm64/features.rst
--- a/Documentation/arch/loongarch/features.rst
+++ b/Documentation/arch/loongarch/features.rst
--- a/Documentation/arch/m68k/features.rst
+++ b/Documentation/arch/m68k/features.rst
--- a/Documentation/arch/mips/features.rst
+++ b/Documentation/arch/mips/features.rst
--- a/Documentation/arch/nios2/features.rst
+++ b/Documentation/arch/nios2/features.rst
--- a/Documentation/arch/openrisc/features.rst
+++ b/Documentation/arch/openrisc/features.rst
--- a/Documentation/arch/parisc/features.rst
+++ b/Documentation/arch/parisc/features.rst
--- a/Documentation/arch/powerpc/features.rst
+++ b/Documentation/arch/powerpc/features.rst
--- a/Documentation/arch/riscv/features.rst
+++ b/Documentation/arch/riscv/features.rst
--- a/Documentation/arch/s390/features.rst
+++ b/Documentation/arch/s390/features.rst
--- a/Documentation/arch/sh/features.rst
+++ b/Documentation/arch/sh/features.rst
--- a/Documentation/arch/sparc/features.rst
+++ b/Documentation/arch/sparc/features.rst
--- a/Documentation/arch/x86/features.rst
+++ b/Documentation/arch/x86/features.rst
--- a/Documentation/arch/xtensa/features.rst
+++ b/Documentation/arch/xtensa/features.rst
--- a/Documentation/sphinx/kernel_feat.py
+++ b/Documentation/sphinx/kernel_feat.py
--- a/Documentation/translations/zh_CN/arch/loongarch/features.rst
+++ b/Documentation/translations/zh_CN/arch/loongarch/features.rst
--- a/Documentation/translations/zh_CN/arch/mips/features.rst
+++ b/Documentation/translations/zh_CN/arch/mips/features.rst
--- a/Documentation/translations/zh_TW/arch/loongarch/features.rst
+++ b/Documentation/translations/zh_TW/arch/loongarch/features.rst
--- a/Documentation/translations/zh_TW/arch/mips/features.rst
+++ b/Documentation/translations/zh_TW/arch/mips/features.rst