fix(syncthing): ensure both config and data dirs exist at startup (fixes #10126) (#10127)

Previously we'd only ensure the config dir, which is often but not
always the same as the data dir.

Fixes #10126
This commit is contained in:
Jakob Borg
2025-05-25 08:10:17 +02:00
committed by GitHub
parent 1a131a56f2
commit 64b5a1b738

View File

@@ -349,10 +349,12 @@ func (options serveOptions) Run() error {
return nil
}
// Ensure that our home directory exists.
if err := syncthing.EnsureDir(locations.GetBaseDir(locations.ConfigBaseDir), 0o700); err != nil {
l.Warnln("Failure on home directory:", err)
os.Exit(svcutil.ExitError.AsInt())
// Ensure that our config and data directories exist.
for _, loc := range []locations.BaseDirEnum{locations.ConfigBaseDir, locations.DataBaseDir} {
if err := syncthing.EnsureDir(locations.GetBaseDir(loc), 0o700); err != nil {
l.Warnln("Failed to ensure directory exists:", err)
os.Exit(svcutil.ExitError.AsInt())
}
}
if options.UpgradeTo != "" {