Уязвимость обхода относительного пути в компоненте Google Storage Apache Camel. Эта проблема затрагивает Apache Camel: с версии 4.0.0 до 4.14.9, с версии 4.15.0 до 4.18.4, с версии 4.19.0 до 4.22.0. Потребитель Camel-Google-Storage загружает объекты Google Cloud Storage в локальную файловую систему, если установлен параметр downloadFileName.
Этот параметр документируется как папка или имя файла, и когда его значение не содержит токена выражения, потребитель создает локальное место назначения, добавляя к нему имя объекта: AssessmentFileExpression устанавливает в заголовке имени файла Exchange имя удаленного объекта и оценивает downloadFileName + "/${file:name}". Токен ${file:name} возвращает заголовок имени файла дословно, в отличие от ${file:onlyname}, который применяет к нему FileUtil.stripPath. Полученная строка была передана непосредственно в новый File(result) и blob.downloadTo(file.toPath()) без лексической нормализации и без проверки того, что пункт назначения оставался внутри настроенного каталога.
Имя объекта не является данными, управляемыми маршрутом: потребитель перечисляет корзину, повторяет каждый возвращенный большой двоичный объект и создает один обмен для каждого объекта из blob.getBlobId().getName() дословно, а опция фильтра, которая могла бы ограничить эти имена, не применяется вообще, если она не была явно установлена. Имена объектов Google Cloud Storage представляют собой непрозрачные ключи UTF-8, которые служба хранит и перечисляет в точности так, как написано, без канонизации на стороне сервера, а косая черта — это всего лишь соглашение об отображении для псевдокаталогов, поэтому ключ, содержащий сегменты родительского каталога, сохраняется при круговом обходе без изменений. Таким образом, имя объекта, содержащее такие сегменты, разрешается в местоположение за пределами настроенного каталога downloadFileName, позволяя любому, кто может повлиять на имена, присутствующие в потребляемом сегменте, заставить Camel создать или перезаписать файл в выбранном им месте с привилегиями процесса Camel.
В зависимости от того, во что процесс может записывать, перезапись файла за пределами каталога загрузки может выйти за рамки потери целостности этого файла. Параметр downloadFileName является обычным потребительским параметром и не содержит маркера безопасности, поэтому ничто не сигнализирует пользователям о том, что его значение не применяется в качестве границы сдерживания. Дефект распространяется только на потребителя; у производителя нет приемника загрузки в файл.
Другие потребители загрузки файлов Camel — Camel-file, Camel-ftp, Camel-SMB, Camel-mina-sftp, Camel-Azure-Files и пути загрузки Azure Storage — уже ограничили свои локальные загрузки настроенным каталогом с помощью проверки границы сегмента пути; Camel-google-storage был оставшимся приемником загрузки из хранилища объектов, не охваченным этой работой. Пользователям рекомендуется выполнить обновление до версии 4.22.0, которая устраняет проблему. Если пользователи участвуют в потоке выпусков 4.14.x LTS, им предлагается выполнить обновление до 4.14.9.
Если пользователи находятся в потоке выпусков 4.18.x, им предлагается выполнить обновление до 4.18.4. Для развертываний, которые не могут обновиться немедленно, задайте для параметра фильтра регулярное выражение, которое принимает только простые имена объектов, состоящие из одного сегмента, чтобы любое имя, содержащее разделитель пути или сегмент родительского каталога, исключалось до создания обмена; обратите внимание, что никакая фильтрация не применяется, если параметр не установлен, и что выражение сопоставляется со всем именем объекта. Альтернативно, укажите для downloadFileName явное выражение, которое не передает удаленный путь, например выражение, построенное на ${file:onlyname}, а не на неявном ${file:name}, учитывая, что downloadFileName, содержащее выражение, рассматривается как контролируемое автором маршрута и на него не распространяется проверка содержания, добавленная в исправлении.
В качестве глубокой защиты рассматривайте имена объектов в любой корзине, доступной для записи извне, как ненадежные входные данные и не извлекайте из них пути к локальной файловой системе.
Показать оригинальное описание (EN)
Relative path traversal vulnerability in Apache Camel Google Storage component. This issue affects Apache Camel: from 4.0.0 before 4.14.9, from 4.15.0 before 4.18.4, from 4.19.0 before 4.22.0. The camel-google-storage consumer downloads Google Cloud Storage objects to the local filesystem when the downloadFileName option is set. That option is documented as a folder or a filename, and when its value contains no expression token the consumer builds the local destination by appending the object name to it: evaluateFileExpression sets the Exchange file-name header to the remote object name and evaluates downloadFileName + "/${file:name}". The ${file:name} token returns the file-name header verbatim, unlike ${file:onlyname}, which applies FileUtil.stripPath to it. The resulting string was passed directly to new File(result) and blob.downloadTo(file.toPath()) with no lexical normalization and no check that the destination stayed inside the configured directory. The object name is not route-controlled data: the consumer lists the bucket, iterates every returned blob and creates one exchange per object from blob.getBlobId().getName() verbatim, and the filter option that could restrict those names is not applied at all unless it has been explicitly set. Google Cloud Storage object names are opaque UTF-8 keys that the service stores and lists exactly as written, with no server-side canonicalization, and a forward slash is only a display convention for pseudo-directories, so a key containing parent-directory segments survives round-tripping intact. An object name containing such segments therefore resolved to a location outside the configured downloadFileName directory, letting anyone able to influence the names present in the consumed bucket cause Camel to create or overwrite a file at a location of their choosing, with the privileges of the Camel process. Depending on what the process can write to, overwriting a file outside the download directory can escalate beyond the loss of integrity of that file. The downloadFileName option is an ordinary consumer parameter and carries no security marker, so nothing signalled to users that its value was not being enforced as a containment boundary. The defect is consumer-only; the producer has no download-to-file sink. Camel's other file-download consumers - camel-file, camel-ftp, camel-smb, camel-mina-sftp, camel-azure-files and the Azure Storage download paths - already constrained their local downloads to the configured directory using a path-segment boundary check; camel-google-storage was the remaining object-store download sink not covered by that work. Users are recommended to upgrade to version 4.22.0, which fixes the issue. If users are on the 4.14.x LTS releases stream, then they are suggested to upgrade to 4.14.9. If users are on the 4.18.x releases stream, then they are suggested to upgrade to 4.18.4. For deployments that cannot upgrade immediately, set the filter option to a regular expression that accepts only simple single-segment object names, so that any name carrying a path separator or a parent-directory segment is excluded before an exchange is created; note that no filtering whatsoever is applied when the option is left unset, and that the expression is matched against the whole object name. Alternatively, give downloadFileName an explicit expression that does not carry the remote path through, for example one built on ${file:onlyname} rather than the implicit ${file:name}, keeping in mind that a downloadFileName containing an expression is treated as route-author-controlled and is not covered by the containment check added in the fix. As defence in depth, treat the object names in any externally writable bucket as untrusted input and do not derive local filesystem paths from them.