6.0.0-beta1
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
7/26/25
H
istory
A
ttachments
C
omment
W
atch
Download
Comment on [#4380] rename_sentmail_monthly doesdn't check for empty folder name
*
Your Email Address
*
Spam protection
Enter the letters below:
__ .__ .___..__ .__. / `[__) _/ [__)[__] \__.| \./__.| \| |
Comment
> In lib/Identity/imp.php, the getAllSentmailfolders() function doesn't > check for empty strings: > > > > include_once 'Horde/Identity.php'; > > $identity = &Identity::singleton(array('imp', 'imp')); > > $folder_list = $identity->getAllSentmailfolders(); > > var_dump($folder_list); > > > > array(1) { > > [0]=> > > string(0) "" > > } > > > > This especially becomes a problem with the rename_sentmail_monthly > task. If a user has set the sent mail folder for an identity to > "None", they are prompted with this: > > > > The current folder(s) "" will be renamed to "". > > > > If the user confirms the task, the resulting call to rename() > effectively traverses the entire IMAP server. The rename() function > searches for all folders matching $old . $namespace_info['delimiter'] > . '*'. If $old == "" and the delimiter is '/', it results in a > listing for '/*'. On a wu-imapd server, this causes a complete > traversal of the IMAP server from the root filesystem. > > > > Rather than try to rename an empty folder name, it seems like the > correct behavior would be to catch empty strings in some/all of > Identity_imp::getAllSentmailfolders(), IMP_Folder::rename(), and > Maintenance_Task_rename_sentmail_monthly::doMaintenance().
Attachment
Watch this ticket
N
ew Ticket
M
y Tickets
S
earch
Q
uery Builder
R
eports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers