GitHub CodeQL reports for overly permissive file permissions
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- python
- Área
- build-system, security
Línea de trabajo
Revisa las llamadas marcadas en gyp/pylib/gyp/common.py:561, gyp/pylib/gyp/flock_tool.py:43 y gyp/pylib/gyp/mac_tool.py:250. Empieza rastreando cómo se usan estos permisos y la umask del proceso y, después, compara el comportamiento con la regla de CodeQL. Se considerará terminado cuando se documente si cada operación es intencionada y se puede suprimir de forma segura, o se identifique un cambio necesario.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
CodeQL has a Python rule with ID py/overly-permissive-file which flags several chmod/open calls in the node-gyp Python code. I would like to confirm that these operations are intentionally more permissive and safe to suppress.
Flagged locations
| File | Line | Code | CodeQL concern |
|---|---|---|---|
gyp/pylib/gyp/common.py |
561 | os.chmod(tool_path, 0o755) |
World readable |
gyp/pylib/gyp/flock_tool.py |
43 | os.open(lockfile, ..., 0o666) |
World readable/writable |
gyp/pylib/gyp/mac_tool.py |
250 | os.open(lockfile, ..., 0o666) |
World readable/writable |
0o755oncommon.py:561is set because you are unsure whether all build processes are run with the same user0o666on lock files inflock_tool.pyandmac_tool.pyis intentional, with umask expected to restrict actual permissions
Could you confirm these permissions are intentional and by design?
- Lenguaje dominante
- Python
- Estrellas
- 10.7k
- Forks
- 1.9k
- Merge medio
- 1 d 4 h
- PR fusionados (30 d)
- 5
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nodejs/node-gyp
-
dependencies github_actions
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
-
MSYS2
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Todos los issues de nodejs/node-gyp
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
stephrobert/dsoxlab#238 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
sublimehq/package_control#1780 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
nwg-piotr/nwg-displays#145 ·