From 8a848303fe0f19fde02793024299356bcdb06048 Mon Sep 17 00:00:00 2001
From: Kevin Kuehler <keur@xcf.berkeley.edu>
Date: Sun, 10 Nov 2019 14:17:55 -0800
Subject: worker/imap: Fix seqMap race condition

When deleting a message, sometimes FetchDirectoryContents will fire.
FetchDirectoryContents will return a smaller set of UIDs since messages
have been deleted. This operation races with fetching from the seqMap in
client.ExpungeUpdate. Only recreate the seqMap if it can grow so that
messages will continue to be expunged.

Signed-off-by: Kevin Kuehler <keur@xcf.berkeley.edu>
---
 worker/imap/open.go | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/worker/imap/open.go b/worker/imap/open.go
index 452c309..0602a7f 100644
--- a/worker/imap/open.go
+++ b/worker/imap/open.go
@@ -40,7 +40,9 @@ func (imapw *IMAPWorker) handleFetchDirectoryContents(
 		}, nil)
 	} else {
 		imapw.worker.Logger.Printf("Found %d UIDs", len(uids))
-		imapw.seqMap = make([]uint32, len(uids))
+		if len(imapw.seqMap) < len(uids) {
+			imapw.seqMap = make([]uint32, len(uids))
+		}
 		imapw.worker.PostMessage(&types.DirectoryContents{
 			Message: types.RespondTo(msg),
 			Uids:    uids,
-- 
cgit 1.4.1-2-gfad0

ut</a> <a href='/danisanti/profani-tty/'>summary</a> <a href='/danisanti/profani-tty/refs/?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>refs</a> <a href='/danisanti/profani-tty/log/configure-debug'>log</a> <a class='active' href='/danisanti/profani-tty/tree/configure-debug?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>tree</a> <a href='/danisanti/profani-tty/commit/configure-debug?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>commit</a> <a href='/danisanti/profani-tty/diff/configure-debug?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>diff</a> <a href='/danisanti/profani-tty/stats/configure-debug'>stats</a></td><td class='form'><form class='right' method='get' action='/danisanti/profani-tty/log/configure-debug'>
<input type='hidden' name='id' value='1f8b1eb740391941e79e1004ad041f8178a2b674'/><select name='qt'>
<option value='grep'>log msg</option>
<option value='author'>author</option>
<option value='committer'>committer</option>
<option value='range'>range</option>
</select>
<input class='txt' type='search' size='10' name='q' value=''/>
<input type='submit' value='search'/>
</form>
</td></tr></table>
<div class='path'>path: <a href='/danisanti/profani-tty/tree/?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>root</a>/<a href='/danisanti/profani-tty/tree/configure-debug?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>configure-debug</a></div><div class='content'>blob: 63679c9806199047641ce9d0be2fa5eb4326327c (<a href='/danisanti/profani-tty/plain/configure-debug?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>plain</a>) (<a href='/danisanti/profani-tty/blame/configure-debug?id=1f8b1eb740391941e79e1004ad041f8178a2b674'>blame</a>)
<table summary='blob content' class='blob'>
<tr><td class='linenumbers'><pre><a id='n1' href='#n1'>1</a>
<a id='n2' href='#n2'>2</a>
<a id='n3' href='#n3'>3</a>
<a id='n4' href='#n4'>4</a>
</pre></td>
<td class='lines'><pre><code>