From b75ea1b05286cd5352a8a4bf35977cd174024acd Mon Sep 17 00:00:00 2001 From: Mikhail Mazurskiy Date: Mon, 15 Jun 2020 22:00:56 +1000 Subject: [PATCH] Fix error check logic If copy finished file (err == nil) then use the error returned from out.Close() --- cluster/gce/gci/configure_helper_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cluster/gce/gci/configure_helper_test.go b/cluster/gce/gci/configure_helper_test.go index 3eba79b810e..6b0bbc44d99 100644 --- a/cluster/gce/gci/configure_helper_test.go +++ b/cluster/gce/gci/configure_helper_test.go @@ -171,7 +171,7 @@ func copyFile(src, dst string) (err error) { } defer func() { cerr := out.Close() - if cerr == nil { + if err == nil { err = cerr } }()