[Bug 261595]

Irving 261595 at bugs.launchpad.net
Thu Aug 8 00:54:01 UTC 2013


Comment on attachment 756486
Write updated popstate in tempfile first, then move over

Review of attachment 756486:
-----------------------------------------------------------------

Thanks so much for the patch; sorry it took so long for me to review it.

I'm OK with either landing this patch as is, or expanding it to refactor
out the other places in mailnews where we create safe, buffered output
streams.

::: mailnews/base/util/nsMsgUtils.cpp
@@ +1438,4 @@
>    return rv;
>  }
>  
> +nsresult MsgNewSafeBufferedFileOutputStream(nsIOutputStream **aResult,

There are a couple of other places in mailnews where we're creating safe
output streams; it would be great to update those to use this helper
function instead of repeating the code, as part of this patch.

-- 
You received this bug notification because you are a member of Mozilla
Bugs, which is subscribed to Mozilla.
https://bugs.launchpad.net/bugs/261595

Title:
  Thunderbird corrupts Mailbox when hd is full

To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/261595/+subscriptions




More information about the Ubuntu-mozillateam-bugs mailing list