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
}