diff options
| author | Dan Winship <danw@src.gnome.org> | 2001-03-07 23:28:33 +0800 |
|---|---|---|
| committer | Dan Winship <danw@src.gnome.org> | 2001-03-07 23:28:33 +0800 |
| commit | 0d719990521adf12e458fb193c679b8378aa3a55 (patch) | |
| tree | 75faf646cca2d31a871ce41362b8a019e65a887b /camel/camel-remote-store.c | |
| parent | 29fa2cd408f90fe49a060645b494a1dfdc56d9b3 (diff) | |
| download | gsoc2013-evolution-0d719990521adf12e458fb193c679b8378aa3a55.tar.gz gsoc2013-evolution-0d719990521adf12e458fb193c679b8378aa3a55.tar.zst gsoc2013-evolution-0d719990521adf12e458fb193c679b8378aa3a55.zip | |
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
Diffstat (limited to 'camel/camel-remote-store.c')
| -rw-r--r-- | camel/camel-remote-store.c | 6 |
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; } |