From 97dec88eab69d0bf3d806bb1f4adf54c1b345f77 Mon Sep 17 00:00:00 2001
From: Joey Hess <joey@kitenet.net>
Date: Mon, 5 Nov 2012 20:15:36 -0400
Subject: fix directory checking to handle mangled directory names

relative, containing ~/ etc

Also, fix parsing of directory name parameter out of git-shell command.
---
 GitAnnexShell.hs | 28 ++++++++++++++++++++++++----
 1 file changed, 24 insertions(+), 4 deletions(-)

(limited to 'GitAnnexShell.hs')

diff --git a/GitAnnexShell.hs b/GitAnnexShell.hs
index ba312c7d1..42841a647 100644
--- a/GitAnnexShell.hs
+++ b/GitAnnexShell.hs
@@ -17,6 +17,7 @@ import Command
 import Annex.UUID
 import qualified Option
 import Fields
+import Utility.UserInfo
 
 import qualified Command.ConfigList
 import qualified Command.InAnnex
@@ -96,7 +97,8 @@ external :: [String] -> IO ()
 external params = do
 	{- Normal git-shell commands all have the directory as their last
 	 - parameter. -}
-	checkDirectory $ lastMaybe params
+	let lastparam = lastMaybe =<< shellUnEscape <$> lastMaybe params
+	checkDirectory lastparam
 	checkNotLimited
 	unlessM (boolSystem "git-shell" $ map Param $ "-c":fst (partitionParams params)) $
 		error "git-shell failed"
@@ -140,12 +142,30 @@ checkDirectory mdir = do
 	v <- getEnv "GIT_ANNEX_SHELL_DIRECTORY"
 	case (v, mdir) of
 		(Nothing, _) -> noop
-		(Just d, Nothing) -> req d
+		(Just d, Nothing) -> req d Nothing
 		(Just d, Just dir)
 			|  d `equalFilePath` dir -> noop
-			| otherwise -> req d
+			| otherwise -> do
+				home <- myHomeDir
+				d' <- canondir home d
+				dir' <- canondir home dir
+				if d' `equalFilePath` dir'
+					then noop
+					else req d' (Just dir')
 	where
-		req d = error $ "Only allowed to access " ++ d
+		req d mdir' = error $ unwords 
+			[ "Only allowed to access"
+			, d
+			, maybe "and could not determine directory from command line" ("not " ++) mdir'
+			]
+
+		{- A directory may start with ~/ or in some cases, even /~/,
+		 - or could just be relative to home, or of course could
+		 - be absolute. -}
+		canondir home d
+			| "~/" `isPrefixOf` d = return d
+			| "/~/" `isPrefixOf` d = return $ drop 1 d
+			| otherwise = relHome $ absPathFrom home d
 
 checkEnv :: String -> IO ()
 checkEnv var = do
-- 
cgit v1.2.3