From 41a82222ccf9aa5f2c5ddbc057cc089a95989288 Mon Sep 17 00:00:00 2001
From: dbadoy4874 <dbadoy4874@gmail.com>
Date: Fri, 19 Aug 2022 22:55:38 +0900
Subject: [PATCH] file_snapshot: do defer after os.Open succeed

---
 file_snapshot.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/file_snapshot.go b/file_snapshot.go
index e4d1ea4f9..ba238f914 100644
--- a/file_snapshot.go
+++ b/file_snapshot.go
@@ -424,11 +424,11 @@ func (s *FileSnapshotSink) Close() error {
 
 	if !s.noSync && runtime.GOOS != "windows" { // skipping fsync for directory entry edits on Windows, only needed for *nix style file systems
 		parentFH, err := os.Open(s.parentDir)
-		defer parentFH.Close()
 		if err != nil {
 			s.logger.Error("failed to open snapshot parent directory", "path", s.parentDir, "error", err)
 			return err
 		}
+		defer parentFH.Close()
 
 		if err = parentFH.Sync(); err != nil {
 			s.logger.Error("failed syncing parent directory", "path", s.parentDir, "error", err)