[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openbsd-tech
Subject:    Re: acme-client: garbage collect unused url in netproc()
From:       Florian Obser <florian () openbsd ! org>
Date:       2024-04-28 9:54:29
Message-ID: m15xw1n76y.fsf () openbsd ! org
[Download RAW message or body]

OK

On 2024-04-28 10:36 +02, Theo Buehler <tb@theobuehler.org> wrote:
> When RFC 8555 support was added in r1.23, token and url were left
> unused. token was removed a day later. url had to wait a bit longer.
>
> Index: netproc.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/acme-client/netproc.c,v
> diff -u -p -r1.34 netproc.c
> --- netproc.c	28 Apr 2024 08:29:56 -0000	1.34
> +++ netproc.c	28 Apr 2024 08:35:19 -0000
> @@ -663,7 +663,7 @@ netproc(int kfd, int afd, int Cfd, int c
>  {
>  	int		 rc = 0;
>  	size_t		 i;
> -	char		*cert = NULL, *thumb = NULL, *url = NULL, *error = NULL;
> +	char		*cert = NULL, *thumb = NULL, *error = NULL;
>  	struct conn	 c;
>  	struct capaths	 paths;
>  	struct order	 order;
> @@ -898,7 +898,6 @@ out:
>  	close(dfd);
>  	close(rfd);
>  	free(cert);
> -	free(url);
>  	free(thumb);
>  	free(c.kid);
>  	free(c.buf.buf);
>

-- 
In my defence, I have been left unsupervised.

[prev in list] [next in list] [prev in thread] [next in thread]