Closed
Bug 497598
Opened 16 years ago
Closed 16 years ago
hang on shutdown appending msg to imap folder via a filter
Categories
(MailNews Core :: Networking: IMAP, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Thunderbird 3.0rc1
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
(Keywords: hang)
Attachments
(2 files)
894 bytes,
patch
|
standard8
:
review+
standard8
:
superreview+
|
Details | Diff | Splinter Review |
871 bytes,
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
I have a filter that copies incoming pop3 mail to an imap folder - if I shutdown at the wrong time, it hangs because the underlying temp file that's getting appended gets removed, and we spin reading 0 bytes from the file input stream.
The underlying file shouldn't be getting removed (rkent is working on fixing that, I think), but we shouldn't be getting 0 bytes back from the file inputstream either, and I think bailing out here is better than spinning.
Attachment #382730 -
Flags: superreview?(bugzilla)
Attachment #382730 -
Flags: review?(bugzilla)
Assignee | ||
Comment 1•16 years ago
|
||
this hits me often enough that it's worth trying to fix.
Flags: blocking-thunderbird3+
Target Milestone: --- → Thunderbird 3.0b4
Updated•16 years ago
|
Attachment #382730 -
Flags: superreview?(bugzilla)
Attachment #382730 -
Flags: superreview+
Attachment #382730 -
Flags: review?(bugzilla)
Attachment #382730 -
Flags: review+
Assignee | ||
Comment 2•16 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: Thunderbird 3.0b4 → Thunderbird 3.0b3
Assignee | ||
Comment 3•16 years ago
|
||
This needs a different fix - Wayne is having this problem in the real world, and I've got a different possible fix.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: Thunderbird 3.0b3 → Thunderbird 3.0rc1
Assignee | ||
Comment 4•16 years ago
|
||
we should be checking for the zero length file before we start issuing protocol.
Assignee | ||
Comment 5•16 years ago
|
||
I've requested a try server build with the above patch so Wayne can see if it fixes his problem.
Comment 6•16 years ago
|
||
try build got me through my problem
Assignee | ||
Updated•16 years ago
|
Attachment #405938 -
Flags: superreview?(neil)
Attachment #405938 -
Flags: review?(neil)
Updated•16 years ago
|
Attachment #405938 -
Flags: superreview?(neil)
Attachment #405938 -
Flags: superreview+
Attachment #405938 -
Flags: review?(neil)
Attachment #405938 -
Flags: review+
Assignee | ||
Comment 7•16 years ago
|
||
second patch checked in.
Status: REOPENED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•