Fix file descriptor leak in GCP FGetObject on error paths

In FGetObject, objectFile is opened but never closed when
NewReader or io.Copy fails, leaking a file descriptor.

Add defer objectFile.Close() after the successful open to ensure
the file is always closed, regardless of which error path is taken.
The existing explicit Close() on the happy path runs first; the
deferred close is a harmless no-op in that case.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>

Author: SebTardif

internal/bucket/gcp/gcp.go

@@ -229,6 +229,7 @@ func (c *GCSClient) FGetObject(ctx context.Context, bucketName, objectName, loca
 	if err != nil {
 		return "", err
 	}
+	defer objectFile.Close()
 
 	// Get Object data.
 	objectReader, err := c.Client.Bucket(bucketName).Object(objectName).If(gcpstorage.Conditions{