InfoQ Homepage News Vulnerability in Java Reflection Library Fixed after 30 Months

# Vulnerability in Java Reflection Library Fixed after 30 Months

In July 2013, security organization Security Explorations discovered a security vulnerability in Java 7u25 by which an attacker could completely escape the Java sandbox. Oracle released a patch in update 7u40, but as Security Explorations announced earlier this year, the patch only addressed the proof of concept, and a simple code modification still exposes the vulnerability. Moreover, subsequent investigation has revealed that the vulnerability is even more severe than initially reported. After the issue became public, Oracle released a patch as part of 8u77.

The vulnerability can be found in the new reflection library, available since Java 7, and more specifically in the new MethodHandle class used for dynamically accessing and invoking methods. It is related to the way classes loaded by different ClassLoaders are treated. Understanding of the issue requires some basic knowledge ofthe way Java ClassLoaders work; since class loading is one of the least understood aspects of Java, we will proceed to provide an introduction to this concept prior to explaining the issue itself.

Java has the ability to load code dynamically at runtime from a variety of sources. This is accomplished through a special category of classes called ClassLoaders. A standard Java implementation might provide several ClassLoaders to load classes from a file system, a URL, or a zipped file, among others, but also provides developers the ability to create their own custom ClassLoaders to handle custom requirements. The usual way to interact with a ClassLoader is by calling its loadClass(String) method, which will accept the name of a class, and either return the associated Class object if found, or throw a ClassNotFoundException otherwise. Every class in a Java application is loaded this way by one ClassLoader or another.

Whenever a Java application is started, the following ClassLoaders will come to effect, in order:

1. Bootstrap ClassLoader: part of the JVM itself, and therefore specific to each JVM implementation. This ClassLoader doesn’t have a parent ClassLoader, and is used to load the core classes under the package java.lang.
2. Extension ClassLoader: responsible for loading classes in extension libraries, which tend to be different on each Java installation. The Extension ClassLoader will load anything located in the path indicated by the variable java.ext.dirs.
3. Application ClassLoader: responsible for loading the main class of the application, together with all the classes in the application classpath.
4. Custom ClassLoader: any other ClassLoader used within the application. It’s optional, and it may not be present depending on the application.

The ability to load classes dynamically at runtime using custom ClassLoaders has created the opportunity for a number of applications that wouldn’t be possible otherwise, but unfortunately it has also created a number of security concerns, particularly around class impersonation. A developer could, in theory, create a custom ClassLoader that loads a compromising implementation of the primordial class java.lang.Object, and use this custom Object in a Java application. This would be a security concern for two reasons: first, this custom Object class would have access to all the package-visibility content of classes in the package java.lang; and second, this custom Object would be treated by the JVM as the standard Object class, and therefore as one of the trusted class that are part of the Java implementation.

In order to secure Java against these vulnerabilitie, Java classes are identified using three attributes: class name, package, and ClassLoader reference. If two different classes with the same name and package were loaded by different ClassLoaders, Java considers them not equal, and attempting to assign between them will cause a ClassCastException. This protects the environment from class impersonation.

## Partial Fix and Resulting Vulnerability

The initial vulnerability as reported by Security Explorations, and categorised as CVE-2013-5838, indicated that, when invoking a method through a Method Handle, the ClassLoader of the class whose method is being invoked was not being checked, meaning an attacker could achieve class impersonation exactly in the way described above.

Code sample where the original vulnerability is shown, the ClassLoader of target class is not checked; source: Security Explorations.

Oracle provided a fix for this in September 2013 as part of Java 7u40 which consisted in a class visibility check that compared the ClassLoaders of the expected type and the incoming type in the following manner:

• If both ClassLoaders are the same, then the two types are fully compatible by definition.
• If one ClassLoader is a parent of the other one, then it is understood that the two classes were loaded through the normal hierarchy of ClassLoaders, and therefore it is safe to assume that they are equivalent.

It is in this second check where Security Explorations found that the exploit is still possible with some minor changes. First, the custom ClassLoader used to impersonate a class can be configured to have the target ClassLoader as its parent, for this is a parameter that can be set through the API:

URLClassLoader lookup_CL = URLClassLoader.newInstance(urlArray, member_CL);

Mechanism through which the custom ClassLoader is perceived as part of the hierarchy; source: Security Explorations.

Resulting code flow after forcing the parent ClassLoader to fail to load the class through a custom-built HTTP server; source Security Explorations.

When the issue resurfaced in March 2016, the latest available version at the time, 8u74, proved to be vulnerable, and Oracle provided a fix for it in 8u77. However, in the release notes of 8u77, the vulnerability was still being referred to as "affecting Java SE running in web browsers on desktops [and] not applicable to Java deployments, typically in servers or standalone desktop applications", while it has been proven to affect server configurations and Google App Engine for Java as well.

## Correction: 29th April 2016

This article incorrectly described the vulnerability as being still available in versions 8u77, 8u91 and 8u92, however, this was fixed in 8u77. The release notes of 8u77 refer to a fix for CVE-2016-0636, which is described as "Unspecified vulnerability [...] via unknown vectors" and doesn't include any obvious references to CVE-2013-5838, the issue described in this article. However, Security Explorations indicates that CVE-2016-0636 is a CVE number for Issue 69, where Issue 69 is their own way to refer to the original CVE-2013-5838.

Style

## Hello stranger!

You need to Register an InfoQ account or or login to post comments. But there's so much more behind being registered.

Get the most out of the InfoQ experience.

Allowed html: a,b,br,blockquote,i,li,pre,u,ul,p

• ##### Smells like clickbait

by Ant hony /

• ##### Re: Smells like clickbait

by Abraham Marin-Perez /

• ##### Re: Smells like clickbait

by Ant hony /

• ##### Re: Smells like clickbait

by Abraham Marin-Perez /

• ##### Smells like clickbait

by Ant hony /

Your message is awaiting moderation. Thank you for participating in the discussion.

Why are you publishing this article if you're unsure whether or not it has been addressed by recent updates? Especially with a title saying it hasn't?

In fact, I'd argue it HAS been fixed in 8u77:

Security Explorations says ( www.security-explorations.com/en/SE-2012-01-sta... ):
24-Mar-2016
- Oracle provides a status report regarding a broken security fix for Issue 69. The company informs that it was fixed by Security Alert for CVE-2016-0636 issued on Mar 23, 2016.

The "Security Alert for CVE-2016-0636" that is mentioned, is available here: www.oracle.com/technetwork/topics/security/aler...

And the release notes of 8u77 ( www.oracle.com/technetwork/java/javase/8u77-rel... ) clearly refer to that exact Security Alert.

Looking forward to hear your counterarguments.

• ##### Re: Smells like clickbait

Your message is awaiting moderation. Thank you for participating in the discussion.

Hello Anthony,

Thanks for your comments. This article refers to vulnerability CVE-2013-5838, while the vulnerability that has been fixed in 8u77 is CVE-2016-0636. One could argue that both codes refer to the same vulnerability, or that the new one is just a way to refer to the modification in the attack that is needed to exploit the vulnerability, but I believe there is no base for that argument for the following reasons:

• On one side, CVE-2016-0636 is described as "Unspecified vulnerability [...] via unknown vectors", which doesn't match the issue at hand (which is perfectly known).
• On the other side, Oracle has specified CVE-2016-0636 as "a vulnerability affecting Java SE running in web browsers on desktops [...] not applicable to Java deployments, typically in servers or standalone desktop applications", however, the vulnerability referred to in this article has been demonstrated to affect also server configurations.

In summary, while it is true that an unspecified vulnerability has been fixed in Java 8u77, that doesn't seem to be the one described here. Given that nothing seems to suggest that the vulnerability has been addressed, the logical conclusion is that it is still present, and therefore I believe the title to be correct.

• ##### Re: Smells like clickbait

by Ant hony /

Your message is awaiting moderation. Thank you for participating in the discussion.

Hi Abraham

However:
CVE-2013-5838 was marked as fixed in 7u45 ( www.oracle.com/technetwork/topics/security/cpuo... )
CVE-2016-0636 was marked as fixed in 8u77 ( www.oracle.com/technetwork/topics/security/aler... )

So which CVE does this article actually refer to? And what about the reference to the Security Explorations page I gave, which states that Oracle has addressed this in CVE-2016-0636?

Either way, it's misleading that the title states it as a fact, while the text says: "judging by their release notes, none of those seems to have addressed the problem".

• ##### Re: Smells like clickbait

Your message is awaiting moderation. Thank you for participating in the discussion.

Hello Anthony,

CVE-2013-5838 was marked as fixed in 7u45, but Security Explorations proved that the fix was only partial. I have to confess though that I had missed the connection between CVE-2013-5838 and CVE-2016-0636 in Security Explorations's website; that is an oversight on my part.

Given that new information, it does seem that the vulnerability has been fixed, thanks for pointing it out. I will proceed to correct the article.

However, I'd like to mention that, given the information that I had at the moment of writing the article, there was no evidence of the issue having been fixed (the public information available on mitre's or Oracle's website wasn't clear enough to deduce that). Therefore, I don't think it is fair to say it was misleading.

Thanks,
Abraham

Allowed html: a,b,br,blockquote,i,li,pre,u,ul,p

Allowed html: a,b,br,blockquote,i,li,pre,u,ul,p

Is your profile up-to-date? Please take a moment to review and update.

Note: If updating/changing your email, a validation request will be sent

Company name:
Company role:
Company size:
Country/Zone:
State/Province/Region:
You will be sent an email to validate the new email address. This pop-up will close itself in a few moments.