On Thu, 2017-02-16 at 16:13 +0100, Christophe Fergeau wrote:
> Less things to keep track of when taking a quick look at the code
> flow
> of the spice_channel_load_ca() method.
>
> Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> ---
> src/spice-channel.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/src/spice-channel.c b/src/spice-channel.c
> index 6556db3..b60b284 100644
> --- a/src/spice-channel.c
> +++ b/src/spice-channel.c
> @@ -2364,10 +2364,6 @@ static gboolean
> spice_channel_delayed_unref(gpointer data)
> static int spice_channel_load_ca(SpiceChannel *channel)
> {
> SpiceChannelPrivate *c = channel->priv;
> - STACK_OF(X509_INFO) *inf;
> - X509_INFO *itmp;
> - X509_STORE *store;
> - BIO *in;
> int i, count = 0;
> guint8 *ca;
> guint size;
> @@ -2382,6 +2378,11 @@ static int spice_channel_load_ca(SpiceChannel
> *channel)
> CHANNEL_DEBUG(channel, "Load CA, file: %s, data: %p", ca_file,
> ca);
>
> if (ca != NULL) {
> + STACK_OF(X509_INFO) *inf;
> + X509_INFO *itmp;
should go to the for block
> + X509_STORE *store;
also used in a different block
> + BIO *in;
> +
> store = SSL_CTX_get_cert_store(c->ctx);
> in = BIO_new_mem_buf(ca, size);
> inf = PEM_X509_INFO_read_bio(in, NULL, NULL, NULL);
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel