Re: [PATCH spice-server] reds: Do not leak SSL context

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 06, 2017 at 04:12:32PM +0000, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
> I can write a test (or better extend the leak one) but
> requires to add ca/certificate/private key.


Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>


> ---
>  server/reds.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/server/reds.c b/server/reds.c
> index 2983aae..fc720a3 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -3703,6 +3703,10 @@ SPICE_GNUC_VISIBLE void spice_server_destroy(SpiceServer *reds)
>      }
>      reds_core_timer_remove(reds, reds->mig_timer);
>  
> +    if (reds->ctx) {
> +        SSL_CTX_free(reds->ctx);
> +    }
> +
>      if (reds->main_dispatcher) {
>          g_object_unref(reds->main_dispatcher);
>      }
> -- 
> 2.9.3
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]