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 Desktop Packages, which is subscribed to thunderbird in Ubuntu. https://bugs.launchpad.net/bugs/261595 Title: Thunderbird corrupts Mailbox when hd is full Status in Mozilla Thunderbird Mail and News: In Progress Status in “thunderbird” package in Ubuntu: Triaged Bug description: Binary package hint: mozilla-thunderbird I fetched the POP3 EMails and the last Message got cut in the middle. Other than in Bug #23302 there was NO warning about little space. Then I got a warning after the error the little space and freed some space up. Now it fetched all allready fetched mails again and added some junk from another email into the unfinished email. In the end this is not so severe, but the check for little space should kick in before and it should not get to corruption of email data. Seen in ubuntu 8.04.1 with thunderbird 2.0.0.16. To manage notifications about this bug go to: https://bugs.launchpad.net/thunderbird/+bug/261595/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp
