From a4364a93d0e80e0d1daa2b28713df05c25970fd1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Schie=C3=9Fle?= <schiessle@owncloud.com>
Date: Mon, 22 Apr 2013 15:29:58 +0200
Subject: [PATCH] delete all share keys if a file gets deleted

---
 apps/files_encryption/lib/keymanager.php | 21 ++++++++++++++++++++-
 apps/files_encryption/lib/proxy.php      | 11 +++--------
 2 files changed, 23 insertions(+), 9 deletions(-)

diff --git a/apps/files_encryption/lib/keymanager.php b/apps/files_encryption/lib/keymanager.php
index 6fb1f128b5..9885f5e550 100755
--- a/apps/files_encryption/lib/keymanager.php
+++ b/apps/files_encryption/lib/keymanager.php
@@ -391,7 +391,26 @@ class Keymanager {
 		return $result;
 		
 	}
-	
+
+	/**
+	 * @brief delete all share keys of a given file
+	 * @param \OC_FilesystemView $view
+	 * @param type $userId owner of the file
+	 * @param type $filePath path to the file, relative to the owners file dir
+	 */
+	public static function delAllShareKeys(\OC_FilesystemView $view, $userId, $filePath) {
+		
+		if ($view->is_dir($userId.'/files/'.$filePath)) {
+			$view->unlink($userId.'/files_encryption/share-keys/'.$filePath);
+		} else {
+			$localKeyPath = $view->getLocalFile($userId.'/files_encryption/share-keys/'.$filePath);
+			$matches = glob(preg_quote($localKeyPath).'*.shareKey');
+			foreach ($matches as $ma) {
+				unlink($ma);
+			}
+		}
+	}
+
 	/**
 	 * @brief Delete a single user's shareKey for a single file
 	 */
diff --git a/apps/files_encryption/lib/proxy.php b/apps/files_encryption/lib/proxy.php
index 505fad440d..bd25465ee6 100644
--- a/apps/files_encryption/lib/proxy.php
+++ b/apps/files_encryption/lib/proxy.php
@@ -256,18 +256,13 @@ class Proxy extends \OC_FileProxy {
 		// Format path to be relative to user files dir
 		$relPath = $util->stripUserFilesPath( $path );
 
-// 		list( $owner, $ownerPath ) = $util->getUidAndFilename( $relPath );
-
-		$fileOwner = \OC\Files\Filesystem::getOwner( $path );
-		$ownerPath = $util->stripUserFilesPath( $path );  // TODO: Don't trust $path, fetch owner path
-
-		$filePath = $fileOwner . '/' . 'files_encryption' . '/' . 'keyfiles' . '/'. $ownerPath;
+ 		list( $owner, $ownerPath ) = $util->getUidAndFilename( $relPath );
 
 		// Delete keyfile & shareKey so it isn't orphaned
 		if (
 			! (
-				Keymanager::deleteFileKey( $view, $fileOwner, $ownerPath )
-				&& Keymanager::delShareKey( $view, $fileOwner, $ownerPath )
+				Keymanager::deleteFileKey( $view, $owner, $ownerPath )
+				&& Keymanager::delAllShareKeys( $view, $owner, $ownerPath )
 			)
 		) {