~sircmpwn/aerc2#505: 
Race condition in textinput.go

Currently there is an innocent use of time.AfterFunc in lib/ui/textinput.go (https://git.sr.ht/~sircmpwn/aerc/tree/548facab/item/lib/ui/textinput.go#L275-277) which is the biggest offender in my logs of Go race detector because AfterFunc runs ti.showCompletions() in a new goroutine, but none of TextInput state is thread-safe, unfortunately i didnt managed to work around an easy fix while preserving the functionality that introduced it (https://git.sr.ht/~sircmpwn/aerc/commit/7160f98a), leting you guys know so if someone with more understading of overall aerc code base can spot an easy fix or work on it. there's another race going on more rarely with pty spawning, but i couldn't figure it out yet.

--wagner

Status
REPORTED
Submitter
~wgr
Assigned to
No-one
Submitted
3 months ago
Updated
3 months ago
Labels
No labels applied.