
CVE ID: CVE-2025-61686
Versiones afectadas: @react-router/node 7.0.0 a 7.9.3
Tipo de vulnerabilidad: Path Traversal / Directory Traversal
La vulnerabilidad se encuentra en la función getFile() y en la lógica relacionada con las operaciones de archivo en el archivo packages/react-router-node/sessions/fileStorage.ts.
En la línea 267 de packages/react-router/lib/server-runtime/sessions.ts:
async getSession(cookieHeader, options) {
let id = cookieHeader && (await cookie.parse(cookieHeader, options));
let data = id && (await readData(id));
return createSession(data || {}, id || "");
}
El Session ID se obtiene de la Cookie mediante el método cookie.parse().
En la función decodeCookieValue() de packages/react-router/lib/server-runtime/cookies.ts:
async function decodeCookieValue(
value: string,
secrets: string[],
): Promise<any> {
if (secrets.length > 0) {
// 如果配置了 secrets,会验证签名
for (let secret of secrets) {
let unsignedValue = await unsign(value, secret);
if (unsignedValue !== false) {
return decodeData(unsignedValue);
}
}
return null; // 签名验证失败返回 null
}
// 如果没有配置 secrets(未签名),直接返回解码后的值
return decodeData(value);
}
Problema clave: cuando la cookie no está firmada (secrets es un array vacío o no está configurado), decodeCookieValue devuelve directamente el valor de la cookie decodificado, y el atacante puede controlar completamente este valor.
En packages/react-router-node/sessions/fileStorage.ts:
export function getFile(dir: string, id: string): string {
// Divide the session id up into a directory (first 2 bytes) and filename
// (remaining 6 bytes) to reduce the chance of having very large directories,
return path.join(dir, id.slice(0, 4), id.slice(4));
}
Esta función divide el session ID en dos partes:
id.slice(0, 4)id.slice(4)Luego se usa path.join() para unir la ruta.
Escenario de ataque: cuando se usa createFileSessionStorage() y la cookie no está firmada:
../../etc/passwdgetFile() procesa:
id.slice(0, 4) = ../.id.slice(4) = /etc/passwdpath.join(dir, ../., /etc/passwd)path.join() normaliza la ruta, si dir ya es una ruta relativa o ha sido procesado, la path traversal aún puede ser posible.Una forma de explotación más precisa:
....//etc/passwd
id.slice(0, 4) = ....id.slice(4) = //etc/passwd/etc/passwdO bien:
../../../etc/passwd (16 caracteres)
id.slice(0, 4) = ../.id.slice(4) = ./etc/passwdpath.join(), podría permitir path traversal.Las siguientes operaciones de archivo pueden verse afectadas:
readData(id) - al leer datos de la sesión
async readData(id) {
try {
let file = getFile(dir, id);
let content = JSON.parse(await fsp.readFile(file, "utf-8"));
// ...
}
}
updateData(id, data, expires) - al actualizar datos de la sesión
async updateData(id, data, expires) {
let content = JSON.stringify({ data, expires });
let file = getFile(dir, id);
await fsp.mkdir(path.dirname(file), { recursive: true });
await fsp.writeFile(file, content, "utf-8");
}
deleteData(id) - al eliminar datos de la sesión
async deleteData(id) {
try {
await fsp.unlink(getFile(dir, id));
}
}
Deben cumplirse todas las siguientes condiciones:
createFileSessionStorage()cookie no se establece secrets o secrets es un array vacío)getSession(cookieHeader)
→ cookie.parse(cookieHeader)
→ decodeCookieValue(value, secrets) // 未签名时直接返回 value
→ readData(id)
→ getFile(dir, id) // 路径拼接,存在路径遍历风险
→ fsp.readFile(file) / fsp.writeFile(file) / fsp.unlink(file)
getFile() no valida ni normaliza el parámetro id.path.join(): aunque path.join() normaliza la ruta, en determinados casos (por ejemplo, cuando la ruta ya contiene .. antes de unirla) aún puede permitir path traversal.