[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 5/7] crypto/tlssession: Introduce qcrypto_tls_creds_check_
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v4 5/7] crypto/tlssession: Introduce qcrypto_tls_creds_check_endpoint() helper |
Date: |
Thu, 17 Jun 2021 10:33:46 +0100 |
User-agent: |
Mutt/2.0.7 (2021-05-04) |
On Wed, Jun 16, 2021 at 09:21:45PM +0200, Philippe Mathieu-Daudé wrote:
> On 6/16/21 9:12 PM, Richard Henderson wrote:
> > 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.
>
> I tried to follow the maintainer/subsystem style (I was also tempted to
> use qcrypto_tls_creds_check_endpoint directly). ui/vnc uses the TLS
> "session" API and not the "creds" one. Daniel, what is your preference?
I think we don't need this extra function - just use the function from
earlier directly.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH v4 2/7] block/nbd: Use qcrypto_tls_creds_check_endpoint(), (continued)
[PATCH v4 7/7] crypto: Make QCryptoTLSCreds* structures private, Philippe Mathieu-Daudé, 2021/06/16
[PATCH v4 6/7] ui/vnc: Use qcrypto_tls_session_check_role(), Philippe Mathieu-Daudé, 2021/06/16