On 6/16/21 9:22 AM, Philippe Mathieu-Daudé wrote:
Introduce the qcrypto_tls_creds_check_endpoint() helper
to avoid accessing QCryptoTLSCreds internal 'endpoint' field
directly.
I don't understand this one. Comment ...
+bool qcrypto_tls_session_check_role(QCryptoTLSCreds *creds,
+ QCryptoTLSCredsEndpoint endpoint,
+ Error **errp)
+{
+ return qcrypto_tls_creds_check_endpoint(creds, endpoint, errp);
+}
... doesn't match the function.
The new function is a pure forwarder, and begs the question of why the caller isn't using
qcrypto_tls_creds_check_endpoint directly.
r~