~rucikir added bug 4 years ago

~gdiazlo 4 years ago

I've been bitten by it too. I was changing accounts when it happened. It does not always happens though.

Sijmen Schoon 4 years ago · edit

(Hopefully I'm doing this correctly, this is my first time doing git related things via email)

I just got this issue too, on 0.4.0. After taking a quick gander at the source code, it seems to be because msg.Envelope is nil on the indicated line (on the current master HEAD (fe1cabb), that's line 108).

I feel this happens when (one of) the messages has not finished loading, but I'm having a hard time reproducing the issue. Perhaps expanding the nil check on line 98 will be enough?

widgets/msglist.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/widgets/msglist.go b/widgets/msglist.go index e38dd9e..c340770 100644 --- a/widgets/msglist.go +++ b/widgets/msglist.go @@ -95,7 +95,7 @@ func (ml *MessageList) Draw(ctx *ui.Context) { break }

•           if msg == nil {
  

•           if msg == nil || msg.Envelope == nil {
                    needsHeaders = append(needsHeaders, uid)
                    ml.spinner.Draw(ctx.Subcontext(0, row, textWidth, 1))
                    row += 1
  

~rucikir 4 years ago

I’ve ran into this issue once again; the trace I’m getting has changed and matches the previous message. I think it happened when one message hasn’t loaded or is being send, and I quit aerc (but I’m not sure).

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x562457762ced]

goroutine 1 [running]:
git.sr.ht/~sircmpwn/aerc/widgets.(*MessageList).Draw(0xc000249da0, 0xc00080e6c0)
	git.sr.ht/~sircmpwn/aerc/widgets/msglist.go:108 +0x62d
git.sr.ht/~sircmpwn/aerc/lib/ui.(*Grid).Draw(0xc000154630, 0xc00080e630)
	git.sr.ht/~sircmpwn/aerc/lib/ui/grid.go:144 +0x277
git.sr.ht/~sircmpwn/aerc/widgets.(*AccountView).Draw(0xc000286bd0, 0xc00080e630)
	git.sr.ht/~sircmpwn/aerc/widgets/account.go:139 +0x3b
git.sr.ht/~sircmpwn/aerc/lib/ui.(*TabContent).Draw(0xc000286b60, 0xc00080e630)
	git.sr.ht/~sircmpwn/aerc/lib/ui/tab.go:394 +0x7c
git.sr.ht/~sircmpwn/aerc/lib/ui.(*Grid).Draw(0xc0001544d0, 0xc000226090)
	git.sr.ht/~sircmpwn/aerc/lib/ui/grid.go:144 +0x277
git.sr.ht/~sircmpwn/aerc/widgets.(*Aerc).Draw(0xc000154580, 0xc000226090)
	git.sr.ht/~sircmpwn/aerc/widgets/aerc.go:176 +0x3f
git.sr.ht/~sircmpwn/aerc/lib/ui.(*UI).Tick(0xc00026c820, 0xc0002c6000)
	git.sr.ht/~sircmpwn/aerc/lib/ui/ui.go:113 +0x1d1
main.main()
	git.sr.ht/~sircmpwn/aerc/aerc.go:194 +0x697

it seems to be because msg.Envelope is nil on the indicated line

https://git.sr.ht/~sircmpwn/aerc/tree/743683a0c19c75ba87d65b6d0c9be117d0224a3e/widgets/msglist.go#L108

Indeed, that seems to be the case.

~y0ast 4 years ago

I hit this too and submitted the relevant patch: https://lists.sr.ht/~sircmpwn/aerc/patches/14540

I got it when quickly switching between two loading folders.

~acksld 3 years ago

This happened to me as well, although I was not interacting at all with aerc. It suddenly just crashed. It seems to have happened just when I received a new email (in fact the verification email for signing up to sourcehut :P ).