Md Mahir Asef Kabir created CLOUDSTACK-10421:
------------------------------------------------
Summary: Usage of Empty TrustManager Methods is insecure
Key: CLOUDSTACK-10421
URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10421
Project: CloudStack
Issue Type: Improvement
Security Level: Public (Anyone can view this level - this is the default.)
Reporter: Md Mahir Asef Kabir
*Vulnerability Description:* In
“plugins/api/vmware-sioc/src/main/java/org/apache/cloudstack/util/vmware/VMwareUtil.java”,
inside private static class TrustAllTrustManager implements TrustManager,
X509TrustManager, the overridden methods have no body -
{code:java}
public void checkServerTrusted(X509Certificate[] certs, String authType) throws
CertificateException
public void checkClientTrusted(X509Certificate[] certs, String authType) throws
CertificateException
{code}
*Reason it’s vulnerable:* If a method responsible for checking certificates
doesn’t have any body, then it will trust all certificates.
*Suggested Fix:* Adding necessary certificate verification logic in the
overridden methods.
*Feedback:* Please select any of the options down below to help us get an idea
about how you felt about the suggestion -
# Liked it and will make the suggested changes
# Liked it but happy with the existing version
# Didn’t find the suggestion helpful
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
