ShellEscape aint always escaping shells

Offen

#19,906 erstellt am 27.6.2025 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 32/100

Issue-Typ: Bug
Klarheit: Muss geklärt werden
Aktivitätsstatus: Veraltet
Bereich: security

Rechercherichtung

Beginne bei ruby/ql/lib/codeql/ruby/security/CommandInjectionCustomizations.qll um Zeile 53 und untersuche, wie shellescape() erkannt wird. Überprüfe, ob eine beliebige benutzerdefinierte Funktion die Sanitizer-Prüfung erfüllen kann, bestimme dann das beabsichtigte Verhalten und wie es abgedeckt werden sollte. Erledigt ist die Aufgabe, wenn die Abfrage eine nicht verwandte Dummy-Funktion nicht mehr als gültige Sanitization behandelt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Ruby

Hey,

Does not this line mean if a user has defined a dummy function called shellescape() would be considered as a valid sanitizer ?

https://github.com/github/codeql/blob/e02affd327603e89519b67ceffc7e60948831cc1/ruby/ql/lib/codeql/ruby/security/CommandInjectionCustomizations.qll#L53

Thank you

Vorherrschende Sprache: CodeQL
Sterne: 10.1k
Forks: 2.1k
Ø Merge: 2 T. 16 Std.
Gemergte PRs (30 T.): 143

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus github/codeql

[aw] Update rust-analyzer has stale lock file Offen

agentic-workflows

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100

github/codeql#22656 · Erstellt am 23.9.2026
JavaScript externs model StorageEvent() with no type argument Offen

false-positive javascript

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

github/codeql#22632 · 1 Kommentar · Erstellt am 20.9.2026
C#: cs/simplifiable-boolean-expression false positive on Nullable<bool> compared with a literal Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100

github/codeql#22556 · Erstellt am 14.9.2026
Add AlertSuppression.ql for Rust (inline // codeql[...] suppression) Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

github/codeql#21637 · 2 Kommentare · Erstellt am 02.4.2026
Organization owned immutable actions raise warning about unpinned 3rd party actions tag Offen

false-positive

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100

github/codeql#21076 · 3 Kommentare · 3 Reaktionen · Erstellt am 21.12.2025

Alle Issues in github/codeql