diff options
| author | Ludovic Courtès <ludo@gnu.org> | 2016-03-17 21:36:22 +0100 |
|---|---|---|
| committer | Ludovic Courtès <ludo@gnu.org> | 2016-03-17 23:53:53 +0100 |
| commit | 14d6ca3e4dd23ee92adb5e2fcf58546e67534631 (patch) | |
| tree | ce6792c9212681e8bd4558ded2debc51aef362ae | |
| parent | 097a951e96718a037dbfa6d579e2d26f7dab3e82 (diff) | |
| download | guix-14d6ca3e4dd23ee92adb5e2fcf58546e67534631.tar.gz guix-14d6ca3e4dd23ee92adb5e2fcf58546e67534631.zip | |
lint: Do not leak file descriptors for TLS connections.
Partially fixes <http://bugs.gnu.org/20145>.
* guix/scripts/lint.scm (probe-uri): Use 'close-connection' instead of
'close-port'.
| -rw-r--r-- | guix/scripts/lint.scm | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/guix/scripts/lint.scm b/guix/scripts/lint.scm index f135bde9df..27b9e155ec 100644 --- a/guix/scripts/lint.scm +++ b/guix/scripts/lint.scm @@ -20,7 +20,7 @@ ;;; along with GNU Guix. If not, see <http://www.gnu.org/licenses/>. (define-module (guix scripts lint) - #:use-module (guix store) + #:use-module ((guix store) #:hide (close-connection)) #:use-module (guix base32) #:use-module (guix download) #:use-module (guix ftp-client) @@ -41,7 +41,8 @@ #:use-module (web uri) #:use-module ((guix build download) #:select (maybe-expand-mirrors - open-connection-for-uri)) + open-connection-for-uri + close-connection)) #:use-module (web request) #:use-module (web response) #:use-module (srfi srfi-1) @@ -296,7 +297,7 @@ for connections to complete; when TIMEOUT is #f, wait as long as needed." (force-output port) (read-response port)) (lambda () - (close port)))) + (close-connection port)))) (case (response-code response) ((301 302 307) |