hardening: audit shell_exec and exec paths for command injection

Offen
#249 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
48/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Ruhig
Tech-Stack
php
Bereich
security

Rechercherichtung

Start by reviewing the listed command-execution sites in flowview_devices.php, flowview_bulkarin.php, and functions.php, tracing how each dynamic value reaches shell_exec or exec. Check whether shell arguments and any pipeline components are fully escaped, and look for places where the shell pipeline could be replaced with direct PHP operations. Done means every listed path is safely handled; the issue does not name tests to run.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement

Findings (validated by xAI Grok security review)

shell_exec/exec with dynamic inputs (HIGH)
  • flowview_devices.php:893,897,901: shell_exec with $safe_port in netstat/ss commands
  • flowview_bulkarin.php:127: exec with whois queries (partially escaped)
  • functions.php:5820: exec with wget command
  • functions.php:6558: shell_exec with flow_export binary

Most paths use escapeshellarg() but some concatenate variables into shell pipeline strings.

Recommended fixes

Audit each exec path for complete escaping. Replace shell pipelines with direct PHP operations where possible.

Vorherrschende Sprache
PHP
Sterne
21
Forks
13
Ø Merge
1 T. 15 Std.
Gemergte PRs (30 T.)
13

Entwicklungsumgebung

  • Kein Dockerfile und keine Docker-Compose-Datei
  • Hat eine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

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

Mehr aus Cacti/plugin_flowview

Alle Issues in Cacti/plugin_flowview

Ähnliche Issues

Weitere Issues zu PHP

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.