Bug 1563665 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

In nsMsgCompose.cpp nsMsgCompose::ProcessReplyFlags(), it handles multiple uri's in a while loop and assumes they are delimited with commas. Therefore, the uri with "com,com" as the folder gets split at the comma resulting in an invalid uri that fails when GetMsgDBHdrFromURI() is called and returns at the NS_ENSURE_SUCCESS() at line 3148. This cause the store of the \Answered imap flag to never be set by the imap protocol code.

At this time, I don't know what the solution is. It seems like everywhere else a folder name with a comma works OK. Can there really be more than one reply at a time?  If so, maybe the delimiter should be something other than a comma, but what?
In nsMsgCompose.cpp nsMsgCompose::ProcessReplyFlags(), it handles multiple uri's in a while loop and assumes they are delimited with commas. Therefore, the uri with "com,com" as the folder gets split at the comma resulting in an invalid uri that fails when GetMsgDBHdrFromURI() is called and returns at the NS_ENSURE_SUCCESS() at line 3148. This cause the store of the \Answered imap flag to never be set by the imap protocol code.

At this time, I don't know what the solution is. It seems like everywhere else a folder name with a comma works OK. Can there really be more than one reply at a time?  If so, maybe the delimiter should be something other than a comma, but what?

Edit: Here's where the error occurs: https://searchfox.org/comm-central/rev/493d9f10ec1cf52cec804decffecc74d32326123/mailnews/compose/src/nsMsgCompose.cpp#3145

Back to Bug 1563665 Comment 3