owncloud driver: Close files before we rename them (cs3org#296)

This fixes renaming on operation systems with file locking
mirekys · Oct 15, 2019 · 3a03735 · 3a03735
1 parent e5837ff
commit 3a03735
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/storage/fs/owncloud/owncloud.go b/pkg/storage/fs/owncloud/owncloud.go
@@ -959,10 +959,10 @@ func (fs *ocFS) Upload(ctx context.Context, ref *storageproviderv0alphapb.Refere
 	if err != nil {
 		return errors.Wrap(err, "ocFS: error creating tmp fn at "+path.Dir(np))
 	}
-	defer tmp.Close()
 	defer os.RemoveAll(tmp.Name())
 
 	_, err = io.Copy(tmp, r)
+	tmp.Close()
 	if err != nil {
 		return errors.Wrap(err, "ocFS: error writing to tmp file "+tmp.Name())
 	}