shithub: hugo

Download patch

ref: 5f5fccbc36c1072150d1b665d9d9ef93a5401b82
parent: 4ddd5361c1ee9b8abbd1f1cb60395d0979127f03
author: Bjørn Erik Pedersen <[email protected]>
date: Sat Jul 30 12:21:57 EDT 2016

Do not return empty theme dirs

This prevents reading data etc. from the root.

Fixes #2320

--- a/helpers/path.go
+++ b/helpers/path.go
@@ -183,13 +183,15 @@
 }
 
 func getThemeDirPath(path string) (string, error) {
-	var themeDir string
-	if ThemeSet() {
-		themeDir = filepath.Join(GetThemeDir(), path)
-		if _, err := os.Stat(themeDir); os.IsNotExist(err) {
-			return "", fmt.Errorf("Unable to find %s directory for theme %s in %s", path, viper.GetString("theme"), themeDir)
-		}
+	if !ThemeSet() {
+		return "", errors.New("No theme set")
 	}
+
+	themeDir := filepath.Join(GetThemeDir(), path)
+	if _, err := os.Stat(themeDir); os.IsNotExist(err) {
+		return "", fmt.Errorf("Unable to find %s directory for theme %s in %s", path, viper.GetString("theme"), themeDir)
+	}
+
 	return themeDir, nil
 }