www

commit b633038bf9245888908fd319ab1c5e336daf4399
parent 22eab3e09d57c378a91d20a5973b4654975d26ea
Author: Dan Stillman <dstillman@zotero.org>
Date:   Fri, 30 Jun 2017 18:08:46 -0400

Close directory iterator in Attachments.hasMultipleFiles()

This was leaving a file descriptor to the directory open during file
syncing.

Diffstat:
M
chrome/content/zotero/xpcom/attachments.js
 | 
4
+++-

1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/chrome/content/zotero/xpcom/attachments.js b/chrome/content/zotero/xpcom/attachments.js
@@ -957,11 +957,13 @@ Zotero.Attachments = new function(){
 			}
 		}
 		catch (e) {
-			iterator.close();
 			if (e != StopIteration) {
 				throw e;
 			}
 		}
+		finally {
+			iterator.close();
+		}
 		return numFiles > 1;
 	});