From a30b310c04579a235be3e2a725445d5418dbff6b Mon Sep 17 00:00:00 2001
From: Sebastian Stenzel <sebastian.stenzel@gmail.com>
Date: Sat, 5 Mar 2016 14:49:46 +0100
Subject: [PATCH] close underlying file, if exception in constructor of
 CryptoReadableFile or CryptoWritableFile

---
 .../filesystem/crypto/CryptoFile.java         | 24 +++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/main/filesystem-crypto/src/main/java/org/cryptomator/filesystem/crypto/CryptoFile.java b/main/filesystem-crypto/src/main/java/org/cryptomator/filesystem/crypto/CryptoFile.java
index 934e31ce3..3f65e845c 100644
--- a/main/filesystem-crypto/src/main/java/org/cryptomator/filesystem/crypto/CryptoFile.java
+++ b/main/filesystem-crypto/src/main/java/org/cryptomator/filesystem/crypto/CryptoFile.java
@@ -35,7 +35,17 @@ class CryptoFile extends CryptoNode implements File {
 	@Override
 	public ReadableFile openReadable() {
 		boolean authenticate = !fileSystem().delegate().shouldSkipAuthentication(toString());
-		return new CryptoReadableFile(cryptor.getFileContentCryptor(), forceGetPhysicalFile().openReadable(), authenticate, this::reportAuthError);
+		ReadableFile physicalReadable = forceGetPhysicalFile().openReadable();
+		boolean success = false;
+		try {
+			final ReadableFile result = new CryptoReadableFile(cryptor.getFileContentCryptor(), physicalReadable, authenticate, this::reportAuthError);
+			success = true;
+			return result;
+		} finally {
+			if (!success) {
+				physicalReadable.close();
+			}
+		}
 	}
 
 	private void reportAuthError() {
@@ -47,7 +57,17 @@ class CryptoFile extends CryptoNode implements File {
 		if (parent.folder(name).exists()) {
 			throw new UncheckedIOException(new FileAlreadyExistsException(toString()));
 		}
-		return new CryptoWritableFile(cryptor.getFileContentCryptor(), forceGetPhysicalFile().openWritable());
+		WritableFile physicalWrtiable = forceGetPhysicalFile().openWritable();
+		boolean success = false;
+		try {
+			final WritableFile result = new CryptoWritableFile(cryptor.getFileContentCryptor(), physicalWrtiable);
+			success = true;
+			return result;
+		} finally {
+			if (!success) {
+				physicalWrtiable.close();
+			}
+		}
 	}
 
 	@Override