fysac/cve-2019-20326Desbordamiento de búfer en el montón en GNOME gThumb y Linux Mint Pix
GNOME gThumb y Linux Mint Pix utilizan el módulo cairo_io para mostrar varios formatos de imagen con la biblioteca gráfica Cairo. En versiones de gThumb < 3.8.3 y Pix < 2.4.5, cairo_io es vulnerable a un desbordamiento del búfer del montón mediante una imagen JPEG con un ancho o alto superior a 32767 píxeles. La vulnerabilidad existe en la función _cairo_image_surface_create_from_jpeg() de extensions/cairo_io/cairo-image-surface-jpeg.c.
cairo_io admite dimensiones JPEG máximas de 32767 x 32767 (CAIRO_MAX_IMAGE_SIZE).1 Al asignar una superficie Cairo, tanto para el ancho como para el alto, _cairo_image_surface_create_from_jpeg() toma el mínimo entre CAIRO_MAX_IMAGE_SIZE y la longitud especificada en el segmento SOF0 del archivo. Por ejemplo, si un archivo JPEG indica tener un tamaño de 40000 x 30000, cairo_io asignará un objeto cairo_surface_t válido solo para 32767 x 30000 píxeles.
En el fragmento siguiente, srcinfo.output_width y srcinfo.output_height son las dimensiones originales de la imagen, mientras que destination_width y destination_height son las dimensiones limitadas. Estas últimas son las que se pasan a _cairo_image_surface_create() para la creación de la superficie.2
_cairo_image_surface_transform_get_steps (CAIRO_FORMAT_ARGB32,
MIN (srcinfo.output_width, CAIRO_MAX_IMAGE_SIZE),
MIN (srcinfo.output_height, CAIRO_MAX_IMAGE_SIZE),
orientation,
&destination_width,
&destination_height,
&line_start,
&line_step,
&pixel_step);
// ...
surface = _cairo_image_surface_create (CAIRO_FORMAT_ARGB32, destination_width, destination_height);
Sin embargo, al escribir los datos de píxeles en la superficie asignada, _cairo_image_surface_create_from_jpeg() itera sobre las dimensiones originales especificadas por el JPEG, srcinfo.output_width y srcinfo.output_height, en lugar de las adecuadas destination_width y destination_height. La estructura de bucles anidados que se muestra a continuación aparece en cinco ubicaciones de cairo-image-surface-jpeg.c, cada una manejando un tipo diferente de espacio de color.3 4 5 6 7
while (srcinfo.output_scanline < srcinfo.output_height) {
// ...
for (x = 0; x < srcinfo.output_width; x++) {
// ...
memcpy (p_surface, &pixel, sizeof (guint32));
Debido a que las dimensiones originales del JPEG pueden ser mayores que el área de memoria designada para la superficie, esta inconsistencia puede permitir que datos de píxeles controlados por el atacante se escriban más allá de los límites del búfer data de la estructura cairo_surface_t.
Vea el caso de prueba minimizado, poc.min.jpg. Este archivo tiene 107 bytes y especifica dimensiones de 1 x 33000.
$ gthumb ./poc.min.jpg
=================================================================
==20729==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x7f49fdc587fc at pc 0x7f4a12200534 bp 0x7f49fe46e240 sp 0x7f49fe46e230
WRITE of size 4 at 0x7f49fdc587fc thread T5 (pool-gthumb)
#0 0x7f4a12200533 in _cairo_image_surface_create_from_jpeg ../extensions/cairo_io/cairo-image-surface-jpeg.c:372
#1 0x55b8e24aad9b in load_image_thread ../gthumb/gth-image-loader.c:241
#2 0x7f4a1a1a5d21 (/lib/x86_64-linux-gnu/libgio-2.0.so.0+0xb2d21)
#3 0x7f4a1a34f853 (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x7b853)
#4 0x7f4a1a34f110 (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x7b110)
#5 0x7f4a19668668 in start_thread /build/glibc-4WA41p/glibc-2.30/nptl/pthread_create.c:479
#6 0x7f4a19590322 in clone (/lib/x86_64-linux-gnu/libc.so.6+0x122322)
0x7f49fdc587fc is located 0 bytes to the right of 131068-byte region [0x7f49fdc38800,0x7f49fdc587fc)
allocated by thread T5 (pool-gthumb) here:
#0 0x7f4a1a658ce6 in calloc (/lib/x86_64-linux-gnu/libasan.so.5+0x10dce6)
#1 0x7f4a18aba5a1 (/lib/x86_64-linux-gnu/libpixman-1.so.0+0x1a5a1)
Thread T5 (pool-gthumb) created by T0 here:
#0 0x7f4a1a585805 in pthread_create (/lib/x86_64-linux-gnu/libasan.so.5+0x3a805)
#1 0x7f4a1a371a16 (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x9da16)
SUMMARY: AddressSanitizer: heap-buffer-overflow ../extensions/cairo_io/cairo-image-surface-jpeg.c:372 in _cairo_image_surface_create_from_jpeg
Shadow bytes around the buggy address:
0x0fe9bfb830a0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0x0fe9bfb830b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0x0fe9bfb830c0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0x0fe9bfb830d0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0x0fe9bfb830e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
=>0x0fe9bfb830f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00[04]
0x0fe9bfb83100: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0fe9bfb83110: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0fe9bfb83120: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0fe9bfb83130: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0fe9bfb83140: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
Shadow byte legend (one shadow byte represents 8 application bytes):
Addressable: 00
Partially addressable: 01 02 03 04 05 06 07
Heap left redzone: fa
Freed heap region: fd
Stack left redzone: f1
Stack mid redzone: f2
Stack right redzone: f3
Stack after return: f5
Stack use after scope: f8
Global redzone: f9
Global init order: f6
Poisoned by user: f7
Container overflow: fc
Array cookie: ac
Intra object redzone: bb
ASan internal: fe
Left alloca redzone: ca
Right alloca redzone: cb
Shadow gap: cc
==20729==ABORTING