[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#39873] [PATCH 3/3] substitute: Close port at the end of http-multip
From: |
Ludovic Courtès |
Subject: |
[bug#39873] [PATCH 3/3] substitute: Close port at the end of http-multiple-get. |
Date: |
Sat, 07 Mar 2020 22:14:03 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Christopher Baines <address@hidden> scribes:
> * guix/scripts/substitute.scm (http-multiple-get): Add close-port call.
> ---
> guix/scripts/substitute.scm | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/guix/scripts/substitute.scm b/guix/scripts/substitute.scm
> index a88cb5bcfe..e3f5837a8e 100755
> --- a/guix/scripts/substitute.scm
> +++ b/guix/scripts/substitute.scm
> @@ -544,6 +544,7 @@ initial connection on which HTTP requests are sent."
> (()
> (match (drop requests processed)
> (()
> + (close-port p)
LGTM!
Did you notice a file descriptor leak somewhere?
Thank you!
Ludo’.