Una mirada más de cerca a la vulnerabilidad de Spring mvcRequestMatcher

El 20 de marzo de 2023, Spring Security Advisories publicó una entrada de blog que hacía referencia a una vulnerabilidad descubierta internamente, CVE-2023-20860. No se reveló información detallada, salvo que se trataba de un problema de control de acceso relacionado con el uso de mvcMatchers. Los desarrolladores de Spring han corregido el problema y se recomienda actualizar la versión.

¿Quiere vivir una experiencia de primera mano? Pruebe la misión aquí.

Como la seguridad es nuestro principal objetivo en Secure Code Warriorhemos decidido profundizar en esta vulnerabilidad de mvcRequestMatchers y averiguar dónde está el problema.

Spring proporciona la interfaz RequestMatcher para determinar si una solicitud coincide con un patrón de ruta. Echa un vistazo al siguiente fragmento de código donde se utiliza el método de ayuda mvcMatchers para registrar los puntos finales junto con sus requisitos de autenticación y autorización. Por ejemplo, podemos ver que sólo los usuarios con el rol ADMIN pueden acceder al endpoint /logs/audit . 

¿MvcMisMatchers?

En Spring, ** es un patrón que permite buscar cualquier número de directorios y subdirectorios en una URL. Por ejemplo, /bankaccount/** coincidiría con todas las URL que empiecen por /bankaccount/, incluidos subdirectorios como /bankaccount/dashboard/settings. 

El patrón * es un patrón que coincide con cualquier URL y tiene exactamente un nivel de subdirectorio. Por ejemplo, /cuentabancaria/* coincidiría con cuenta bancaria/cuadro de mandos.

Al configurar los matchers con *, Spring afirma que se produjo "un desajuste en la concordancia de patrones entre Spring Security y Spr ing MVC", creando la vulnerabilidad.

Esencialmente, debido a la falta de un separador delante del doble comodín, la ruta no coincide con una solicitud entrante, ya que todas las solicitudes entrantes van precedidas de una barra. Esto significa que las reglas de control de acceso no se aplican y permiten a cualquier usuario no autenticado acceder a los recursos.

Echemos un vistazo al commit que ha solucionado el problema.

El cambio más destacado e importante es la adición de la línea 315, que corrige la omisión de las reglas de autorización y autenticación. Garantiza que cualquier patrón de ruta que se envíe vaya precedido de una barra oblicua (/). 

404 coincidencia no encontrada

Al enviar una petición web a /cuentasbancarias/vista , el método match analizará y comparará los patrones definidos en el filtro de seguridad con la ruta solicitada. El analizador sintáctico convertirá el patrón dado en un árbol de elementos de ruta.

El analizador lee el primer carácter como un SeparatorPathElement. A continuación, continúa leyendo los caracteres de la cadena hasta el siguiente separador, creando un nuevo LiteralPathElement.

Entonces, ¿dónde falla cuando se utiliza ** como patrón? 

Aunque existen muchos tipos de elementos de ruta, los más interesantes son WildcardPathElementy WildcardTheRestPathElement, con sus respectivas representaciones de cadena: * y /**. 

Un WildcardPathElement coincide con cero o más caracteres dentro de un único segmento de ruta, mientras que un WildcardTheRestPathElement coincide con cero o más segmentos de ruta por sí solos (incluidos los separadores).

Esto último nos da una pista de lo que falla al enviar ** como patrón. Durante el análisis busca patrones, pero ** no empieza con la barra oblicua esperada. Así que en lugar de convertirse en un WildcardTheRestPathElement, se convierte en dos WildcardPathElements consecutivos.

A continuación, el patrón analizado se utiliza para comparar con la URL solicitada. Se espera que las rutas empiecen por una barra oblicua, pero los comodines no coinciden con los separadores.

Esto significa que en lugar de un RequestMatchResult, se devuelve un null. En consecuencia, las reglas de control de acceso establecidas en este comparador no se aplicarán a la URL solicitada.

Spring ha solucionado el problema añadiendo una barra. En otras palabras, cualquier patrón ** se convierte en /**, lo que significa que puede ser analizado como un WildcardTheRestPathElement, y se devolverá un RequestMatchResult ya que el patrón ahora coincide con la URL solicitada.

¿Vulnerabilidad o uso indebido de la API?

Es discutible si esto debería considerarse una vulnerabilidad, ya que el código funciona como se pretende. El problema radica básicamente en el hecho de que la documentación de Spring no menciona explícitamente que las rutas deban comenzar con un separador. Por lo tanto, podría considerarse más un caso de mal uso de la API que un error o vulnerabilidad.