Merge pull request #27 from gitopia/escape-path

Escape double quotes in all path names

Author: harry-hov

utils/git.go

@@ -22,7 +22,7 @@ func GetFullCommitSha(repoPath, shortID string) (string, error) {
 
 // LastCommitForPath returns the last commit which modified path for given revision.
 func LastCommitForPath(repoPath, revision string, path string) (string, error) {
-	cmd := exec.Command("bash", "-c", "git log --pretty=%H --max-count=1 "+revision+" -- \""+path+"\"")
+	cmd := exec.Command("bash", "-c", "git log --pretty=%H --max-count=1 "+revision+" -- \""+strings.ReplaceAll(path, "\"", "\\\"")+"\"")
 	cmd.Dir = repoPath
 	out, err := cmd.Output()
 	if err != nil {
@@ -34,7 +34,7 @@ func LastCommitForPath(repoPath, revision string, path string) (string, error) {
 
 // CommitHistory returns the commit history for given revision or path.
 func CommitHistory(repoPath, revision string, path string, offset int, limit int) ([]string, error) {
-	cmd := exec.Command("bash", "-c", "git log --pretty=%H --max-count="+strconv.Itoa(limit)+" --skip="+strconv.Itoa(offset)+" "+revision+" -- "+path)
+	cmd := exec.Command("bash", "-c", "git log --pretty=%H --max-count="+strconv.Itoa(limit)+" --skip="+strconv.Itoa(offset)+" "+revision+" -- \""+strings.ReplaceAll(path, "\"", "\\\"")+"\"")
 	cmd.Dir = repoPath
 	out, err := cmd.Output()
 	if err != nil {
@@ -50,7 +50,7 @@ func CommitHistory(repoPath, revision string, path string, offset int, limit int
 
 // CountCommits returns total count of commits.
 func CountCommits(repoPath, revision string, path string) (string, error) {
-	cmd := exec.Command("bash", "-c", "git rev-list --count "+revision+" -- "+path)
+	cmd := exec.Command("bash", "-c", "git rev-list --count "+revision+" -- \""+strings.ReplaceAll(path, "\"", "\\\"")+"\"")
 	cmd.Dir = repoPath
 	out, err := cmd.Output()
 	if err != nil {