Martchus
/
syncthing
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Data race: broken locking on model.folderIgnores

This commit is contained in:
Jakob Borg 2014-11-29 22:29:49 +01:00
parent a918aa97d9
commit 7f829f0159
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
internal/model/model.go
View File

@ -779,10 +779,12 @@ func (m *Model) GetIgnores(folder string) ([]string, []string, error) {
lines = append(lines, strings.TrimSpace(scanner.Text()))
}
m.fmut.RLock()
var patterns []string
if matcher := m.folderIgnores[folder]; matcher != nil {
patterns = matcher.Patterns()
}
m.fmut.RUnlock()
return lines, patterns, nil
}
@ -1050,7 +1052,7 @@ func (m *Model) ScanFolderSub(folder, sub string) error {
return errors.New("invalid subpath")
}
m.fmut.RLock()
m.fmut.Lock()
fs, ok := m.folderFiles[folder]
dir := m.folderCfgs[folder].Path
@ -1066,7 +1068,8 @@ func (m *Model) ScanFolderSub(folder, sub string) error {
CurrentFiler: cFiler{m, folder},
IgnorePerms: m.folderCfgs[folder].IgnorePerms,
}
m.fmut.RUnlock()
m.fmut.Unlock()
if !ok {
return errors.New("no such folder")
}