github / github/codeql

ShellEscape aint always escaping shells

Aperta
#19,906 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Ruby
Lingua principale
CodeQL
Stelle
10.1k
Fork
2.1k
Merge medio
2g 15h
PR unite (30g)
141

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Start with ruby/ql/lib/codeql/ruby/security/CommandInjectionCustomizations.qll around line 53 and inspect how shellescape() is recognized. Verify whether an arbitrary user-defined function can satisfy the sanitizer check, then determine the intended behavior and how it should be covered. Done means the query no longer treats an unrelated dummy function as valid sanitization.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Ambito
security
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
32/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.