Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Apache Commons Collections to v3.2.2 #121

Merged
merged 1 commit into from
Mar 8, 2016

Conversation

stefano-maggiolo
Copy link
Contributor

Version 3.2.1 has a CVSS 10.0 vulnerability. That's the worst kind of
vulnerability that exists. By merely existing on the classpath, this
library causes the Java serialization parser for the entire JVM process
to go from being a state machine to a turing machine. A turing machine
with an exec() function!

https://commons.apache.org/proper/commons-collections/security-reports.html
http://foxglovesecurity.com/2015/11/06/what-do-weblogic-websphere-jboss-jenkins-opennms-and-your-application-have-in-common-this-vulnerability/

Review on Reviewable

Version 3.2.1 has a CVSS 10.0 vulnerability. That's the worst kind of
vulnerability that exists. By merely existing on the classpath, this
library causes the Java serialization parser for the entire JVM process
to go from being a state machine to a turing machine. A turing machine
with an exec() function!

https://commons.apache.org/proper/commons-collections/security-reports.html
http://foxglovesecurity.com/2015/11/06/what-do-weblogic-websphere-jboss-jenkins-opennms-and-your-application-have-in-common-this-vulnerability/
@CLAassistant
Copy link

CLA assistant check
You should sign our Contributor License Agreement in order to get your pull request merged.

@stefano-maggiolo
Copy link
Contributor Author

I'd prefer not to sign the CLA, but the patch is trivial so no need to take "my" version of the patch.

agapple added a commit that referenced this pull request Mar 8, 2016
Upgrade Apache Commons Collections to v3.2.2
@agapple agapple merged commit 6156981 into alibaba:master Mar 8, 2016
@agapple
Copy link
Member

agapple commented Mar 8, 2016

tks

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.

3 participants