[jboss-jira] [JBoss JIRA] (WFLY-8107) Coverity static analysis, Unwritten field, EntitySaslClient.clientCertUrl (Elytron)

[Ilia Vassilev (JIRA)]

     [ https://issues.jboss.org/browse/WFLY-8107?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ilia Vassilev reassigned WFLY-8107:
-----------------------------------

    Assignee: Ilia Vassilev  (was: Darran Lofthouse)


> Coverity static analysis, Unwritten field, EntitySaslClient.clientCertUrl (Elytron)
> -----------------------------------------------------------------------------------
>
>                 Key: WFLY-8107
>                 URL: https://issues.jboss.org/browse/WFLY-8107
>             Project: WildFly
>          Issue Type: Bug
>          Components: Security
>            Reporter: Martin Choma
>            Assignee: Ilia Vassilev
>
> Coverity found field {{EntitySaslClient.clientCertUrl}} is never filled. So probably initially intended behavior in {{X509Certificate getClientCertificate()}} method is not covered.
> {code:java}
>     private X509Certificate getClientCertificate() throws SaslException {
>         if ((clientCertChain != null) && (clientCertChain.length > 0)) {
>             return clientCertChain[0];
>         } else if (clientCertUrl != null) {
>             try {
>                 return EntityUtil.getCertificateFromUrl(clientCertUrl);
>             } catch (IOException e) {
>                 throw log.mechUnableToObtainServerCertificate(getMechanismName(), clientCertUrl.toString(), e).toSaslException();
>             }
>         } else {
>             throw log.mechCallbackHandlerNotProvidedServerCertificate(getMechanismName()).toSaslException();
>         }
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)