From 892892df9cb04c0a1aee47e0e11f2205836b1e7b Mon Sep 17 00:00:00 2001 From: Not Zed Date: Wed, 1 Aug 2001 06:46:39 +0000 Subject: [PATCH] If we fail to get the lock, make sure we close our locking fd, and reset 2001-08-01 Not Zed * providers/local/camel-mbox-folder.c (mbox_lock): If we fail to get the lock, make sure we close our locking fd, and reset it. Fixes the crash part of 5095. --- camel/ChangeLog | 4 ++++ camel/providers/local/camel-mbox-folder.c | 8 +++++++- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/camel/ChangeLog b/camel/ChangeLog index 3adfa5a..cf15b16 100644 --- a/camel/ChangeLog +++ b/camel/ChangeLog @@ -1,5 +1,9 @@ 2001-08-01 Not Zed + * providers/local/camel-mbox-folder.c (mbox_lock): If we fail to + get the lock, make sure we close our locking fd, and reset it. + Fixes the crash part of 5095. + * providers/imap/camel-imap-folder.c (camel_imap_folder_changed): Slightly re-arranged filter/change notification logic. If we;re filtering, freeze the folder till we're finished to smooth the ui diff --git a/camel/providers/local/camel-mbox-folder.c b/camel/providers/local/camel-mbox-folder.c index 1bab553..ccdce9c 100644 --- a/camel/providers/local/camel-mbox-folder.c +++ b/camel/providers/local/camel-mbox-folder.c @@ -153,7 +153,13 @@ static int mbox_lock(CamelLocalFolder *lf, CamelLockType type, CamelException *e return -1; } - return camel_lock_folder(lf->folder_path, mf->lockfd, type, ex); + if (camel_lock_folder(lf->folder_path, mf->lockfd, type, ex) == -1) { + close(mf->lockfd); + mf->lockfd = -1; + return -1; + } + + return 0; } static void mbox_unlock(CamelLocalFolder *lf) -- 2.7.4