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

Possible CrySL rules to implement from CryptoGuard Crypto-API Benchmark #40

Closed
10 tasks
enriozuni opened this issue May 20, 2019 · 2 comments
Closed
10 tasks
Assignees

Comments

@enriozuni
Copy link
Contributor

enriozuni commented May 20, 2019

CryptoGuard Crypto-API Benchmark reports misuses that are not found from static analysis tools such as CogniCrypt. In their list of tests, the following CrySL rules are not implemented and therefore headless tests could not be done.

Reference of the issue is in this link

Java classes that are used in the CryptoGuard tests and are not yet implemented as CrySL rules can be found below.

Checklist:

  • javax.net.ssl.X509TrustManager
  • javax.net.ssl.HostnameVerifier
  • javax.net.ssl.SSLSession
  • javax.net.ssl.HttpsURLConnection
  • javax.net.ssl.SSLSocket
  • javax.net.ssl.SSLSockerFactory
  • java.security.cert.CertificateException
  • java.security.cert.X509Certificate
  • java.net.URL
  • java.net.MalformedURLException
@kruegers
Copy link
Member

I suspect these will become relevant when you work on this issue.

@AndreSonntag
Copy link
Contributor

javax.net.ssl.X509TrustManager: no security risk
javax.net.ssl.SSLSession: no security risk
java.net.URL: no security risk
javax.net.ssl.HttpsURLConnection: no security risk
java.security.cert.X509Certificate: no security risk

javax.net.ssl.HostnameVerifier: Interface -> just one method, no implementing classes

javax.net.ssl.SSLSocket: rule already exsists
javax.net.ssl.SSLSockerFactory: rule already exsists

Exceptions are not considered

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

No branches or pull requests

3 participants