CVE-2026-31706

In the Linux kernel, the following vulnerability has been resolved: ksmbd: validate num_aces and harden ACE walk in smb_inherit_dacl() smb_inherit_dacl() trusts the on-disk num_aces value from the parent directory's DACL xattr and uses it to size a heap allocation: aces_base = kmalloc(sizeof(struct smb_ace) * num_aces * 2, ...); num_aces is a u16 read from le16_to_cpu(parent_pdacl->num_aces) without checking that it is consistent with the declared pdacl_size. An authenticated client whose parent directory's security.NTACL is tampered (e.g. via offline xattr corruption or a concurrent path that bypasses parse_dacl()) can present num_aces = 65535 with minimal actual ACE data. This causes a ~8 MB allocation (not kzalloc, so uninitialized) that the subsequent loop only partially populates, and may also overflow the three-way size_t multiply on 32-bit kernels. Additionally, the ACE walk loop uses the weaker offsetof(struct smb_ace, access_req) minimum size check rather than the minimum valid on-wire ACE size, and does not reject ACEs whose declared size is below the minimum. Reproduced on UML + KASAN + LOCKDEP against the real ksmbd code path. A legitimate mount.cifs client creates a parent directory over SMB (ksmbd writes a valid security.NTACL xattr), then the NTACL blob on the backing filesystem is rewritten to set num_aces = 0xFFFF while keeping the posix_acl_hash bytes intact so ksmbd_vfs_get_sd_xattr()'s hash check still passes. A subsequent SMB2 CREATE of a child under that parent drives smb2_open() into smb_inherit_dacl() (share has "vfs objects = acl_xattr" set), which fails the page allocator: WARNING: mm/page_alloc.c:5226 at __alloc_frozen_pages_noprof+0x46c/0x9c0 Workqueue: ksmbd-io handle_ksmbd_work __alloc_frozen_pages_noprof+0x46c/0x9c0 ___kmalloc_large_node+0x68/0x130 __kmalloc_large_node_noprof+0x24/0x70 __kmalloc_noprof+0x4c9/0x690 smb_inherit_dacl+0x394/0x2430 smb2_open+0x595d/0xabe0 handle_ksmbd_work+0x3d3/0x1140 With the patch applied the added guard rejects the tampered value with -EINVAL before any large allocation runs, smb2_open() falls back to smb2_create_sd_buffer(), and the child is created with a default SD. No warning, no splat. Fix by: 1. Validating num_aces against pdacl_size using the same formula applied in parse_dacl(). 2. Replacing the raw kmalloc(sizeof * num_aces * 2) with kmalloc_array(num_aces * 2, sizeof(...)) for overflow-safe allocation. 3. Tightening the per-ACE loop guard to require the minimum valid ACE size (offsetof(smb_ace, sid) + CIFS_SID_BASE_SIZE) and rejecting under-sized ACEs, matching the hardening in smb_check_perm_dacl() and parse_dacl(). v1 -> v2: - Replace the synthetic test-module splat in the changelog with a real-path UML + KASAN reproduction driven through mount.cifs and SMB2 CREATE; Namjae flagged the kcifs3_test_inherit_dacl_old name in v1 since it does not exist in ksmbd. - Drop the commit-hash citation from the code comment per Namjae's review; keep the parse_dacl() pointer.
Configurations

Configuration 1 (hide)

OR cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*
cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*
cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*

History

11 Nov 2026, 21:10

Type Values Removed Values Added
Summary
  • (es) En el kernel de Linux, la siguiente vulnerabilidad ha sido resuelta: ksmbd: validar num_aces y reforzar el recorrido de ACE en smb_inherit_dacl() smb_inherit_dacl() confía en el valor num_aces en disco del xattr DACL del directorio padre y lo usa para dimensionar una asignación de pila: aces_base = kmalloc(sizeof(struct smb_ace) * num_aces * 2, ...); num_aces es un u16 leído de le16_to_cpu(parent_pdacl -> num_aces) sin verificar que sea consistente con el pdacl_size declarado. Un cliente autenticado cuyo security.NTACL del directorio padre ha sido manipulado (por ejemplo, mediante corrupción de xattr fuera de línea o una ruta concurrente que omite parse_dacl()) puede presentar num_aces = 65535 con datos ACE reales mínimos. Esto causa una asignación de ~8 MB (no kzalloc, por lo tanto no inicializada) que el bucle subsiguiente solo rellena parcialmente, y también puede desbordar la multiplicación size_t de tres vías en kernels de 32 bits. Además, el bucle de recorrido de ACE usa la verificación de tamaño mínimo más débil offsetof(struct smb_ace, access_req) en lugar del tamaño mínimo válido de ACE en la red, y no rechaza los ACEs cuyo tamaño declarado está por debajo del mínimo. Reproducido en UML + KASAN + LOCKDEP contra la ruta de código real de ksmbd. Un cliente mount.cifs legítimo crea un directorio padre sobre SMB (ksmbd escribe un xattr security.NTACL válido), luego el blob NTACL en el sistema de archivos de respaldo es reescrito para establecer num_aces = 0x FFFF mientras mantiene intactos los bytes posix_acl_hash para que la verificación de hash de ksmbd_vfs_get_sd_xattr() aún pase. Un SMB2 CREATE subsiguiente de un hijo bajo ese padre lleva a smb2_open() a smb_inherit_dacl() (el recurso compartido tiene ’vfs objects = acl_xattr’ configurado), lo que hace fallar al asignador de páginas: WARNING: mm/page_alloc.c:5226 at __alloc_frozen_pages_noprof+0x46c/0x9c0 Workqueue: ksmbd-io handle_ksmbd_work __alloc_frozen_pages_noprof+0x46c/0x9c0 ___kmalloc_large_node+0x68/0x130 __kmalloc_large_node_noprof+0x24/0x70 __kmalloc_noprof+0x4c9/0x690 smb_inherit_dacl+0x394/0x 2430 smb2_open+0x 595d/0x abe0 handle_ksmbd_work+0x3d3/0x 1140 Con el parche aplicado, la protección añadida rechaza el valor manipulado con -EINVAL antes de que se ejecute cualquier asignación grande, smb2_open() recurre a smb2_create_sd_buffer(), y el hijo se crea con un SD predeterminado. Sin advertencia, sin error. Solución mediante: 1. Validar num_aces contra pdacl_size usando la misma fórmula aplicada en parse_dacl(). 2. Reemplazar el kmalloc(sizeof * num_aces * 2) en bruto con kmalloc_array(num_aces * 2, sizeof(...)) para una asignación segura contra desbordamientos. 3. Ajustar la protección del bucle por ACE para requerir el tamaño mínimo válido de ACE (offsetof(smb_ace, sid) + CIFS_SID_BASE_SIZE) y rechazar los ACEs de tamaño insuficiente, coincidiendo con el refuerzo en smb_check_perm_dacl() y parse_dacl(). v1 -mayor que v2: - Reemplazar el error del módulo de prueba sintético en el registro de cambios con una reproducción de ruta real de UML + KASAN impulsada a través de mount.cifs y SMB2 CREATE; Namjae señaló el nombre kcifs3_test_inherit_dacl_old en v1 ya que no existe en ksmbd. - Eliminar la cita del hash de commit del comentario del código según la revisión de Namjae; mantener el puntero de parse_dacl().

30 Oct 2026, 13:18

Type Values Removed Values Added
References
  • () https://git.kernel.org/stable/c/4d37830c2c834987de329d58fb9a319067400361 -

21 Oct 2026, 13:18

Type Values Removed Values Added
References
  • () https://git.kernel.org/stable/c/ad7d18df7423c522ed9dc7e33bd5d54c66b1bc37 -

Information

Published : 2026-05-01 14:16

Updated : 2026-11-11 21:10


NVD link : CVE-2026-31706

Mitre link : CVE-2026-31706

CVE.ORG link : CVE-2026-31706


JSON object : View

Products Affected

linux

  • linux_kernel