GHSA-hm49-wcqc-g2xg net-imap vulnerable to command Injection via "raw" arguments to multiple commands
SummarySeveral `Net::IMAP` commands accept a raw string argument that is sent to the server without validation or escaping.  If this string is derived from user-controlled input, it may contain contain `CRLF` sequences, which an attacker can use to inject arbitrary IMAP commands.Details`Net::IMAP`'s generic argument handling, used by most command arguments, interprets string arguments as an IMAP `astring`.  Depending on the string contents and the connection's UTF-8 support, this encodes strings as either a `atom`, `quoted`, or `literal`.  These are safe from command or argument injection.But the following commands transform specific String arguments to `Net::IMAP::RawData`, which bypasses normal argument validation and encoding and prints the string directly to the socket:`#uid_search`, `#search`, `#uid_sort`, `#sort`, `#uid_thread`, `#thread`when `criteria` is a String, it is sent raw`#uid_fetch`, `#fetchwhen `attr` is a String, it is sent rawwhen `attr` is an Array, each String in `attr` is sent raw`#uid_store`, `#store`when `attr` is a String, it is sent raw`#setquota`:`limit` is interpolated with `#to_s` and that string is sent rawBecause these string arguments are sent without any neutralization, they serve as a direct vector for command splitting.  Any user controlled data interpolated into these strings can be used to break out of the intended command context.Using \"raw data\" arguments for  `#uid_store`, `#store`, and `#setquota` I  both inappropriate and unnecessary.  `Net::IMAP`'s generic argument handling is sufficient to safely validate and encode their arguments.  Users of the library probably do not expect arguments to these commands to be sent raw and might not be wary of passing unvalidated input.The API for search criteria and fetch attributes is intentionally low-level and \"close to the wire\".  It allows developers to use some IMAP extensions without requiring explicit support from the library and allows developers to use complex IMAP grammar without complex argument translation.  Even so, basic validation is appropriate and could neutralize command injection.Although this was explicitly documented for search `criteria`, it was insufficiently documented for fetch `attr`.  So developers may not have realized that the `attr` argument to `#fetch` and `#uid_fetch` is sent as \"raw data\".ImpactIf a developer passes an unvalidated user-controlled input for one of these method arguments, an attacker can append CRLF sequence followed by a new IMAP command (like DELETE mailbox).  Although this does not _directly_ enable data exfiltration, it could be combined with other attack vectors or knowledge of the target system's attributes, e.g.: shared mail folders or the application's installed response handlers.The SEARCH, STORE, and FETCH commands, and their UID variants are some of the most commonly used features of the library.  Applications that build search queries or fetch attributes dynamically based on user input (e.g., mail clients or archival tools) may be at significant risk.The SORT and THREAD commands and their UID variants also handle their search criteria argument similarly to SEARCH and are subject to the same risk.Expected use of `Net::IMAP#setquota` is much more limited: `SETQUOTA` is often only usable by users with special administrative privileges.  Depending on the server, quota administration might be managed through server configuration rather than via the IMAP protocol `SETQUOTA` command.  It is expected to be uncommonly used in system administration scripts or in interactive sessions, it should be completely controlled by trusted users, and should only use trusted inputs.  Calling `#setquota` with untrusted user input is expected to be a very uncommon use case.  Please note however this might be combined with other attacks, for example CSRF, which provide unauthorized access to trusted inputs, and may specifically target users or scripts with administrator privileges.MitigationUpdate to a patched version of `net-imap` which:validates that `Net::IMAP::RawData` is composed of well-formed IMAP `text`, `literal`, and `literal8` values, with no unescaped `NULL`, `CR`, or `LF` bytes.does not use `Net::IMAP::RawData` for `#store`, `#uid_store`, or `#setquota`.Prefer to send search criteria as an array of key value pairs.  Avoid sending it as an interpolated string.If an immediate upgrade is not possible:String inputs to search criteria and fetch attributes can be validated against command injection by checking for `\r` and `\n` characters.Hard-coding the store `attr` argument is often appropriate.  Alternatively, user controlled inputs can be restricted to a small enumerated list which is valid for the calling application.Use `Kernel#Integer` to coerce and validate user controlled inputs to `#setquota` limit.
CVSS v4.0 score: 5.8