Command Injection in Ruby

Learn how to fix Command Injection (CWE-78) in Ruby. Side-by-side vulnerable vs secure code examples for Open3.capture2() and system() array arguments.

💡 Explication en Langage Simple (ELI5)

Imaginez demander à un assistant de bureau d'imprimer un document nommé 'rapport.pdf'. L'injection de commande se produit lorsque quelqu'un fournit le nom 'rapport.pdf; whoami', et l'assistant transmet aveuglément le billet au terminal, imprimant le rapport et lisant le badge d'administrateur.

Concepts Clés et Termes

Web Application Security: Composant d'architecture clé affecté par CWE-918.
CWE-918: Classification standard Common Weakness Enumeration (CWE) pour command-injection-ruby.
Defense-in-Depth: Vérification d'ingénierie multicouche et isolation des limites à l'exécution.

Déroulement de l'Attaque Étape par Étape

Step 1

Ingestion de Paramètres Non Fiables

L'application reçoit des noms d'hôte de diagnostic ou des noms de fichiers directement à partir d'une requête HTTP.

Step 2

Interpolation Shell Non Sécurisée

Le backend assemble une commande shell par simple concaténation de chaînes au lieu d'utiliser un tableau d'arguments isolé.

Step 3

Injection de Séparateur de Commandes

L'attaquant injecte des métacaractères shell comme ';', '&&', '|' ou des accents graves (ex. '127.0.0.1; id') pour échapper au contexte initial.

Step 4

Exécution en Sous-shell et Compromission

Le shell du système d'exploitation exécute la commande injectée avec tous les privilèges du processus du serveur web.

Code Source : Vulnérable vs Sécurisé

✕ IMPLÉMENTATION VULNÉRABLE

# Backtick execution invokes /bin/sh
def fetch_git_log(branch)
  # Input: "main; whoami"
  `git log #{branch}`
end

✓ PATCH SÉCURISÉ ET ROBUSTE

# Multiple arguments to system/Open3 bypass the shell
require 'open3'

def fetch_git_log(branch)
  # branch is passed as an isolated argument to git
  stdout, stderr, status = Open3.capture3("git", "log", branch)
  stdout
end

Liste de Contrôle de Sécurité pour l'Ingénierie

✓Utilisez toujours Open3.capture3("cmd", *args) with discrete arguments.
✓Ne jamais use open() on user-supplied filenames; use File.open().
✓Exécutez Brakeman in your GitHub Actions workflow.