diff options
author | René 'Necoro' Neumann <necoro@necoro.eu> | 2020-04-23 23:47:05 +0200 |
---|---|---|
committer | René 'Necoro' Neumann <necoro@necoro.eu> | 2020-04-23 23:47:05 +0200 |
commit | 6911562cf4214531343b7509afe77e38b28a0801 (patch) | |
tree | 46601b14d15b781c373a99f32880334b10092c68 /internal/imap/mailboxes.go | |
parent | c1fbae6e0f5981d4d60f5b614d5d195ae5cde4a8 (diff) | |
download | feed2imap-go-6911562cf4214531343b7509afe77e38b28a0801.tar.gz feed2imap-go-6911562cf4214531343b7509afe77e38b28a0801.tar.bz2 feed2imap-go-6911562cf4214531343b7509afe77e38b28a0801.zip |
Fix concurrent access to the same folder
Diffstat (limited to 'internal/imap/mailboxes.go')
-rw-r--r-- | internal/imap/mailboxes.go | 41 |
1 files changed, 36 insertions, 5 deletions
diff --git a/internal/imap/mailboxes.go b/internal/imap/mailboxes.go index d0fdede..f1dc6c6 100644 --- a/internal/imap/mailboxes.go +++ b/internal/imap/mailboxes.go @@ -7,8 +7,38 @@ import ( ) type mailboxes struct { - mb map[string]*imap.MailboxInfo - mu sync.RWMutex + mb map[string]*imap.MailboxInfo + mu sync.RWMutex + changeLocks map[string]chan struct{} +} + +func (mbs *mailboxes) unlocking(elem Folder) { + mbs.mu.Lock() + defer mbs.mu.Unlock() + + ch, ok := mbs.changeLocks[elem.str] + if !ok { + panic("Unlocking where nothing is locked") + } + close(ch) + delete(mbs.changeLocks, elem.str) +} + +func (mbs *mailboxes) locking(elem Folder) bool { + mbs.mu.Lock() + ch, ok := mbs.changeLocks[elem.str] + if !ok { + ch = make(chan struct{}) + mbs.changeLocks[elem.str] = ch + mbs.mu.Unlock() + // we created the lock, we are in charge and done here + return false + } else { + // someone else is working, we wait till he's done + mbs.mu.Unlock() // we are not doing anything... + <-ch + return true + } } func (mbs *mailboxes) contains(elem Folder) bool { @@ -28,7 +58,8 @@ func (mbs *mailboxes) add(elem *imap.MailboxInfo) { func NewMailboxes() *mailboxes { return &mailboxes{ - mb: map[string]*imap.MailboxInfo{}, - mu: sync.RWMutex{}, + mb: map[string]*imap.MailboxInfo{}, + changeLocks: map[string]chan struct{}{}, + mu: sync.RWMutex{}, } -}
\ No newline at end of file +} |