Open
Bug 1965376
Opened 12 days ago
Updated 11 days ago
nsIMsgPluggableStore.deleteFolder() should probably fail if there are child folders.
Categories
(MailNews Core :: General, task)
MailNews Core
General
Tracking
(Not tracked)
NEW
People
(Reporter: benc, Unassigned)
References
(Blocks 1 open bug)
Details
nsIMsgPluggableStore.deleteFolder() implementations currently just do a recursive delete on the filesystem.
I think this is dangerous, as it allows for not-yet-deleted child folders to have their files destroyed underneath them.
It'd be better if this method failed if there were child folders. And we force the caller to delete all child folders before parents can be deleted. (which is likely the case already, but who can tell?)
|
Reporter | |
Comment 1•12 days ago
|
||
Also, probably rename to deleteStoreFolder() for grepability reasons.
You need to log in
before you can comment on or make changes to this bug.
Description
•