Skip to content

Conversation

@smolnar82
Copy link
Contributor

What changes were proposed in this pull request?

Per CVE-2018-5968
FasterXML jackson-databind through 2.8.11 and 2.9.x through 2.9.3 allows unauthenticated remote code execution because of an incomplete fix for the CVE-2017-7525 and CVE-2017-17485 deserialization flaws. This is exploitable via two different gadgets that bypass a blacklist.

How was this patch tested?

After updating the affected pom.xml files I've done the following:

1.) Checking Maven's dependency resolution:

[INFO] --- maven-dependency-plugin:2.8:tree (default-cli) @ ambari-server ---
[INFO] org.apache.ambari:ambari-server:jar:2.6.1.0.0
[INFO] +- com.fasterxml.jackson.core:jackson-annotations:jar:2.9.4:compile
[INFO] \- com.fasterxml.jackson.core:jackson-databind:jar:2.9.4:compile
[INFO]    \- com.fasterxml.jackson.core:jackson-core:jar:2.9.4:compile

2.) I executed mvn clean install in utility and in ambari-server:

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 33:54 min
[INFO] Finished at: 2018-02-23T13:56:04+01:00
[INFO] Final Memory: 188M/1638M
[INFO] ------------------------------------------------------------------------

3.) In addition to this; I replaced the content of usr/lib/ambari-server in my vagrant host with the content from ambari-server/target/ambari-server-2.6.0.0.0-dist/usr/lib/ambari-server (where the affected JAR were replaced with version 2.9.4) and restarted the server; logged in and did some actions (in this case I created a cluster); there were no any issues.

…d jackson-core) to 2.9.4 due to security concerns
@smolnar82
Copy link
Contributor Author

@rlevas @adoroszlai @zeroflag @oleewere Please review this PR; thanks!

@asfgit
Copy link

asfgit commented Feb 23, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/783/
Test PASSed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants