Return the return value of camel_service_connect rather than returning

	* camel-remote-store.c (camel_remote_store_connected): Return the
	return value of camel_service_connect rather than returning
	!camel_exception_is_set(), since that will be wrong if ex==NULL...
	Fixes a crash in the IMAP keepalive code, but this same problem
	probably exists in other places too...

svn path=/trunk/; revision=8580

1 files changed, 2 insertions, 4 deletions

diff --git a/camel/camel-remote-store.c b/camel/camel-remote-store.c
index 41e16a27fc..991e5b0e1a 100644
--- a/camel/camel-remote-store.c
+++ b/camel/camel-remote-store.c
@@ -635,9 +635,7 @@ camel_remote_store_refresh_folders (CamelRemoteStore *store, CamelException *ex)
 gboolean
 camel_remote_store_connected (CamelRemoteStore *store, CamelException *ex)
 {
-	if (store->istream == NULL) {
-		camel_service_connect (CAMEL_SERVICE (store), ex);
-		return !camel_exception_is_set (ex);
-	}
+	if (store->istream == NULL)
+		return camel_service_connect (CAMEL_SERVICE (store), ex);
 	return TRUE;
 }