From 286caed92beb805947be348b3dff18e24b52b42c Mon Sep 17 00:00:00 2001 From: Dave Henderson Date: Mon, 23 Aug 2021 21:47:22 -0400 Subject: Fix a few filehandle leaks (#1199) Signed-off-by: Dave Henderson --- data/datasource_file.go | 2 ++ data/datasource_file_test.go | 1 + 2 files changed, 3 insertions(+) (limited to 'data') diff --git a/data/datasource_file.go b/data/datasource_file.go index a94944cd..175f92f0 100644 --- a/data/datasource_file.go +++ b/data/datasource_file.go @@ -51,6 +51,8 @@ func readFile(source *Source, args ...string) ([]byte, error) { return nil, errors.Wrapf(err, "Can't open %s", p) } + defer f.Close() + b, err := ioutil.ReadAll(f) if err != nil { return nil, errors.Wrapf(err, "Can't read %s", p) diff --git a/data/datasource_file_test.go b/data/datasource_file_test.go index 6cbcacd7..aa51dbca 100644 --- a/data/datasource_file_test.go +++ b/data/datasource_file_test.go @@ -21,6 +21,7 @@ func TestReadFile(t *testing.T) { _, _ = f.Write(content) _, _ = fs.Create("/tmp/partial/bar.txt") _, _ = fs.Create("/tmp/partial/baz.txt") + _ = f.Close() source := &Source{Alias: "foo", URL: mustParseURL("file:///tmp/foo")} source.fs = fs -- cgit v1.2.3