Saving preferences fails sometimes

VERIFIED FIXED

Status

()

Core
XPCOM
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: mkaply, Assigned: mkaply)

Tracking

Trunk
x86
OS/2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
We found that there were cases when creating the backup copy of 
prefs.js was failing with TOO_MANY_FILES so we were running out of 
filehandles.

Patch attached to check for that specific case and to up the number of 
file handles at startup. This was recommended by the OS/2 kernel guy.
(Assignee)

Comment 1

16 years ago
Created attachment 91531 [details] [diff] [review]
Fix

Check for the out of handles error around DosCopy, and add an increase of the
file handles in nsAppRunner.

Comment 2

16 years ago
Comment on attachment 91531 [details] [diff] [review]
Fix

looks okay.  I have not tested it nor can.

r=dougt
Attachment #91531 - Flags: review+
(Assignee)

Comment 3

16 years ago
Comment on attachment 91531 [details] [diff] [review]
Fix

I'm going to mark this sr=blizzard since all the change is in OS/2 #ifdef or
OS/2 files.
Attachment #91531 - Flags: superreview+

Comment 4

16 years ago
Comment on attachment 91531 [details] [diff] [review]
Fix

a=asa (on behalf of drivers) for checkin to 1.1
Attachment #91531 - Flags: approval+
(Assignee)

Comment 5

16 years ago
fix checked into branch and trunk. Branch approval via emal from chofmann.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Keywords: fixed1.0.1
Resolution: --- → FIXED
(Assignee)

Comment 6

16 years ago
Mass verify. These are fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.