~sircmpwn/aerc

1687e558d3fae4d5622677c031bb9365a8c3e261 — Reto Brunner 2 years ago 60c5a82 
notmuch/maildir: remove double emit of the dirinfo

There was some bug which could be worked around by double emitting an event.
However that proofed to be brittle:

We send the first message here from the worker goroutine:
a5553438/item/worker/maildir/worker.g=">https://git.sr.ht/~sircmpwn/aerc/tree/a5553438/item/worker/maildir/worker.g=
o#L306

Then Tick() is waked in the main goroutine and calls ProcessMessage:
a5553438/item/widgets/account.go#L100">https://git.sr.ht/~sircmpwn/aerc/tree/a5553438/item/widgets/account.go#L100

ProcessMessage in the main goroutine reads types.Message state with
msg.getId() and msg.InResponseTo():
a5553438/item/worker/types/worker.go#=">https://git.sr.ht/~sircmpwn/aerc/tree/a5553438/item/worker/types/worker.go#=
L74-76

Meanwhile in the worker goroutine we call PostMessage for a second
time with a pointer that points to the *same* previous message that
ProcessMessage is reading:
a5553438/item/worker/maildir/worker.g=">https://git.sr.ht/~sircmpwn/aerc/tree/a5553438/item/worker/maildir/worker.g=
o#L306

The second PostMessage call makes writes to message while
ProcessMessage in the main goroutine is possibly reading:
a5553438/item/worker/types/worker.go#=">https://git.sr.ht/~sircmpwn/aerc/tree/a5553438/item/worker/types/worker.go#=
L59

This led to a data race in the event loop

Reported-By: Wagner Riffel <w@104d.net>
patch browse .tar.gz 
2 files changed, 3 insertions(+), 7 deletions(-)

M worker/maildir/worker.go
M worker/notmuch/worker.go

M worker/maildir/worker.go => worker/maildir/worker.go +0 -2
@@ 299,12 299,10 @@ func (w *Worker) handleOpenDirectory(msg *types.OpenDirectory) error {
		return fmt.Errorf("could not clean directory: %v", err)
	}

	// TODO: why does this need to be sent twice??
	info := &types.DirectoryInfo{
		Info: w.getDirectoryInfo(msg.Directory),
	}
	w.worker.PostMessage(info, nil)
	w.worker.PostMessage(info, nil)
	return nil
}



M worker/notmuch/worker.go => worker/notmuch/worker.go +3 -5
@@ 251,8 251,6 @@ func (w *worker) handleOpenDirectory(msg *types.OpenDirectory) error {
		return err
	}
	info.Message = types.RespondTo(msg)
	//TODO: why does this need to be sent twice??
	w.w.PostMessage(info, nil)
	w.w.PostMessage(info, nil)
	w.done(msg)
	return nil


@@ 507,9 505,9 @@ func (w *worker) loadExcludeTags(
		return nil
	}
	excludedTags := strings.Split(raw, ",")
    for idx, tag := range excludedTags {
        excludedTags[idx] = strings.Trim(tag, " ")
    }
	for idx, tag := range excludedTags {
		excludedTags[idx] = strings.Trim(tag, " ")
	}
	return excludedTags
}