hardening: audit shell_exec and exec paths for command injection

Abierto
#249 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
48/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
php
Área
security

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
PHP
Estrellas
21
Forks
13
Merge medio
1 d 15 h
PR fusionados (30 d)
13

Preparar el entorno

  • Sin Dockerfile ni archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Cacti/plugin_flowview

Todos los issues de Cacti/plugin_flowview

Issues similares

Más issues de PHP

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.