58721 - Checking messages every x minutes hangs Mozilla.

Status: VERIFIED FIXED

(MailNews Core :: Networking, defect, P2)

Description

[hirata masakazu]

I have been experiencing this in a number of past builds, and I can not find a
similar one in Bugzilla.
 MacTrunk 2000103120.
 From Mail/News account settings, "Check for new messages every x minutes" is
set to 60 in POP3 server account. (anything longer than default seem to cause
this bug.)
 Your Password for this POP account is not remembered by Mozilla.
 Quit Mozilla.
 Launch Mozilla.
 Start Mail.
 Hangs.

 The only solution is to force quit Mozilla, and then to edit the entry in
prefs.js manually back to 10 minutes, because the settings is not accessible
from Browser. (This accessibility problem must have been filed as a bug
somewhere, I guess.)

Comment 1

[Asa Dotzler [:asa]]

Confirmed with 110104 Mozilla trunk build on OS9.  This inly seems to happen if
the POP account is your default or only account.  

Comment 2

[Christopher Blizzard (:blizzard)]

Nominating for 0.8

Comment 3

[scottputterman]

nominating means putting in the keywords, not the target milestone.  Reassigning
to racham. I've nominated for nsbeta1 which will put it on the radar.  I think
this is a problem that we know about with biff where certain values overflow
whatever type of int we're using and causes it to act like a negative number and
hence makes biff go off constantly, causing the hang.

Comment 4

[hirata masakazu]

*** Bug 61989 has been marked as a duplicate of this bug. ***

Comment 5

[hirata masakazu]

Since there is a dupe reported for Windows, I am changing this for All/All.

Comment 6

[scottputterman]

I disagree that this is critical for 0.8.  But I agree that this should get
fixed. So I'm putting it in 0.9.

Comment 7

[hirata masakazu]

Then UI must be disabled, because the preference item is not accessible from
Browser window. Should I file another bug for that one?

Comment 8

[racham]

*** Bug 70491 has been marked as a duplicate of this bug. ***

Comment 9

[laurel]

*** Bug 70490 has been marked as a duplicate of this bug. ***

Comment 10

[racham]

This is a data overflow problem. Operation (60000000 * biffInterval), where
biffInterval is the number of minutes user enters in the UI, gave back negative
number when the biffInterval is more than 35. That caused an infinite loop as
the bifftime was always less than currenttime, if the biffInterval > 35 minutes.
So, We need to use nsInt64 var here so that we don't overflow and do the right
thing. Patch coming up.

Comment 11

[racham]

Attachment: patch

Comment 12

[racham]

Adding Gayatri and Seth for reviews.

Comment 13

[gayatrib]

r=gayatrib

Comment 14

[(not reading, please use [e-mail] instead)]

sr=sspitzer

just curious, how big would biffInterval have to be now to cause the same problem?

Comment 15

[racham]

I tried 999999 minutes...didn't get any error. I will try with the higher
intervals later and update the bug and see if the problem comesback..

Getting this in now. thanks.

Comment 16

[racham]

Done.

Comment 17

[racham]

*** Bug 61914 has been marked as a duplicate of this bug. ***

Comment 18

[Sheela Ravindran]

verified for biff set to 60 minutes the application does not freeze
2001-04-04-12 win98
2001-04-03-10 liux
2001-04-03-09 mac