From 7184ec28cced142c655c06d360250c7fb330809c Mon Sep 17 00:00:00 2001 From: Sebastian Spaeth Date: Sun, 8 Jan 2012 12:26:47 +0100 Subject: [PATCH] Sanity check return value of UIDVALIDTY response We have a reported case where response('UIDVALIDITY') returned [None] which results in an ugly non-intuitive crash. Sanity check and report something nicer. Signed-off-by: Sebastian Spaeth --- offlineimap/folder/IMAP.py | 1 + 1 file changed, 1 insertion(+) diff --git a/offlineimap/folder/IMAP.py b/offlineimap/folder/IMAP.py index d23fb5f..f04f871 100644 --- a/offlineimap/folder/IMAP.py +++ b/offlineimap/folder/IMAP.py @@ -71,6 +71,7 @@ class IMAPFolder(BaseFolder): # SELECT receives UIDVALIDITY response self.selectro(imapobj) typ, uidval = imapobj.response('UIDVALIDITY') + assert uidval != [None], "response('UIDVALIDITY') returned [None]!" return long(uidval[0]) finally: self.imapserver.releaseconnection(imapobj)