[Commits] [SCM] claws branch, gtk3, updated. 3.16.0-123-ge4274cf

[ticho at claws-mail.org]

The branch, gtk3 has been updated
       via  e4274cfcd78d8039995c4f897e3f900ac3c7987e (commit)
       via  ad6d9254091e1c5249d8f70f8b73d67e5a45e3e7 (commit)
      from  1b5888c7bd5bf06759282b18eeaeaad934ca7914 (commit)

Summary of changes:
 src/etpan/nntp-thread.c |    2 ++
 src/imap.c              |   12 ++++++++++--
 2 files changed, 12 insertions(+), 2 deletions(-)


- Log -----------------------------------------------------------------
commit e4274cfcd78d8039995c4f897e3f900ac3c7987e
Author: Andrej Kacian <ticho at claws-mail.org>
Date:   Wed Mar 14 21:35:05 2018 +0100

    Handle IMAP search failure instead of ignoring it.
    
    Error out on protocol error during search, disconnect
    the IMAP session, and display an error dialog about it.
    
    Closes bug 3986 - IMAP quick search using non-ASCII
    characters creates an infinite loop

diff --git a/src/imap.c b/src/imap.c
index 5c99688..8d0ded9 100644
--- a/src/imap.c
+++ b/src/imap.c
@@ -2468,7 +2468,12 @@ static gint	search_msgs		(Folder			*folder,
 	if (progress_cb)
 		progress_cb(progress_data, TRUE, container->total_msgs, 0, container->total_msgs);
 
-	if (result == MAILIMAP_NO_ERROR) {
+	if (result == MAILIMAP_ERROR_PROTOCOL) {
+		debug_print("search_msgs - got protocol error, aborting\n");
+		imap_handle_error(SESSION(session), NULL, result);
+		alertpanel_error_log(_("Search failed due to server error."));
+		return -1;
+	} if (result == MAILIMAP_NO_ERROR) {
 		gint result = 0;
 
 		*msgs = imap_uid_list_from_lep(uidlist, &result);
@@ -2489,9 +2494,12 @@ static gint	search_msgs		(Folder			*folder,
 		if (IMAP_FOLDER(folder)->search_charset_supported)
 			return search_msgs(folder, container, msgs, on_server, predicate,
 				   progress_cb, progress_data);
-		else
+		else {
+			imap_handle_error(SESSION(session), NULL, result);
 			return -1;
+		}
 	} else {
+		imap_handle_error(SESSION(session), NULL, result);
 		return -1;
 	}
 }

commit ad6d9254091e1c5249d8f70f8b73d67e5a45e3e7
Author: wwp <wwp at free.fr>
Date:   Wed Mar 14 11:22:47 2018 +0100

    Clear statusbar's progressbar in case of NNTP xhdr/xover failure.

diff --git a/src/etpan/nntp-thread.c b/src/etpan/nntp-thread.c
index 26079eb..3c147d2 100644
--- a/src/etpan/nntp-thread.c
+++ b/src/etpan/nntp-thread.c
@@ -867,6 +867,7 @@ int nntp_threaded_xover(Folder * folder, guint32 beg, guint32 end, struct newsnn
 			if (l != NULL)
 				newsnntp_xover_resp_list_free(l);
 			newsnntp_xover_resp_list_free(h);
+			statusbar_progress_all(0, 0, 0);
 			return result.error;
 		}
 
@@ -965,6 +966,7 @@ int nntp_threaded_xhdr(Folder * folder, const char *header, guint32 beg, guint32
 			if (l != NULL)
 				newsnntp_xhdr_free(l);
 			newsnntp_xhdr_free(h);
+			statusbar_progress_all(0, 0, 0);
 			return result.error;
 		}
 

-----------------------------------------------------------------------


hooks/post-receive
-- 
Claws Mail