Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

[SECURITY] - An attacker can write files with exact names to arbitrary locations on the filesystem

Abierto Apto para principiantes
#714 5 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
72/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
c
Área
desktop, security

Línea de trabajo

Lee shell/ev-window.c alrededor de las líneas 7374 y 7400, y sigue las rutas de guardado de adjuntos locales y remotos. Comprueba cómo se manejan los nombres de los adjuntos en cada ruta; el trabajo estará terminado cuando al guardar un adjunto no se pueda usar ../ para salir del directorio elegido, preservando el nombre de archivo previsto.

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

Descripción

Description

Xreader 4.6.3 (commit ef5a50d) passes the raw PDF attachment filename directly to g_file_get_child() when saving attachments via "Save Attachment As...". Since g_file_get_child() resolves ../ sequences, an attacker-controlled filename can escape the user-selected target directory and write files to arbitrary locations with the exact filename specified by the attacker — no random suffix.

This is the most critical of the attachment vulnerabilities, as it enables writing .desktop files (or any other file) with precise names to arbitrary locations, leading to arbitrary code execution.

Affected Code

shell/ev-window.c:7374:

save_to = g_file_get_child(target_file, ev_attachment_get_name(attachment));

shell/ev-window.c:7400:

dest_file = g_file_get_child(target_file, ev_attachment_get_name(attachment));

Both the local and remote save paths use the unsanitized attachment name.

Steps to Reproduce

  1. Generate a malicious PDF using the PoC script from the full report — the attachment is named ../Desktop/test.desktop and contains a valid .desktop entry that executes an arbitrary command
  2. Open poc.pdf in xreader
  3. In the sidebar, right-click the attachment and select "Save Attachment As..."
  4. Select any directory (e.g., ~/Documents/)
  5. test.desktop is written to ~/Desktop/ instead of the selected directory
  6. Double-clicking the .desktop file executes the embedded command

Suggested Fix

Extract the basename before passing to g_file_get_child() in both code paths:

// line 7374
-save_to = g_file_get_child(target_file, ev_attachment_get_name(attachment));
+gchar *basename = g_path_get_basename(ev_attachment_get_name(attachment));
+save_to = g_file_get_child(target_file, basename);
+g_free(basename);

// line 7400
-dest_file = g_file_get_child(target_file, ev_attachment_get_name(attachment));
+gchar *basename = g_path_get_basename(ev_attachment_get_name(attachment));
+dest_file = g_file_get_child(target_file, basename);
+g_free(basename);

Impact

An attacker can write files with exact names to arbitrary locations on the filesystem. By writing a .desktop file to ~/Desktop/, arbitrary code execution is achieved when the user double-clicks it. The only user interaction required is opening the PDF, right-clicking the attachment, selecting "Save As", and choosing any directory.

References

Lenguaje dominante
C
Estrellas
285
Forks
82
Métricas de merge de PR
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 linuxmint/xreader

Todos los issues de linuxmint/xreader

Issues similares

Más issues de C

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.