266327 - [BEOS] Saving internal files with nsLocalFileUnix::CreateUnique fails because of missing check for NS_EXCL in bfile.c

Status: RESOLVED FIXED

(NSPR :: NSPR, defect)

Description

[tqh]

User-Agent:       Mozilla/5.0 (BeOS; U; BeOS BePC; en-US; rv:1.7.3) Gecko/20041026 Firefox/1.0
Build Identifier: Mozilla/5.0 (BeOS; U; BeOS BePC; en-US; rv:1.7.3) Gecko/20041026 Firefox/1.0

Recent builds on BeOS after 0.93 complains when trying to save preferences with
the 'Can't save preferences'.. error message, but the changes are saved. 

It doesn't matter if it's a new profile. It has popped up with all people that
tried a recent build.

Reproducible: Always
Steps to Reproduce:
1. Open preferences
2. Change values
3. Close preferences

Actual Results:  
'Can't save preferences'-message shows up, but prefs are saved.

Expected Results:  
prefs are saved.

Comment 1

[Niels Reedijk]

Attachment: The patch

This patch fixes bug 266327 and 266328 . And it keeps firefox from spilling out
a lot of information on cookies that are probably lost (which aren't).

Comment 2

[Niels Reedijk]

Changed the summary and component of the bug. Bug 266328 is in fact a dupicate
of this one. 

Comment 3

[Niels Reedijk]

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

Comment 4

[Simon]

Comment on attachment 165205 [details] [diff] [review]
The patch

Looks good to me.

Sometimes it's easier to get checkin if you only change the needed bit - ie
just add the one case needed and remove the formatting changes. But I'll give
it a review anyway.

I'm not sure of the process we have to follow to get something checked into
NSPR.

Comment 5

[Simon]

Comment on attachment 165205 [details] [diff] [review]
The patch

Forgot to add
r=[e-mail] to the above comment

Comment 6

[Niels Reedijk]

Comment on attachment 165205 [details] [diff] [review]
The patch

Asking super-review from the NSPR module owner. This is a BeOS only-change and
will not affect other platforms in any way. The patch does touch some
formatting, but not the functionality of those lines. It ends up being
identical to unix.c (in regards to the flag checking)

Comment 7

[Wan-Teh Chang]

I checked in this fix on the NSPR trunk (NSPR 4.6).

Checking in bfile.c;
/cvsroot/mozilla/nsprpub/pr/src/md/beos/bfile.c,v  <--  bfile.c
new revision: 3.13; previous revision: 3.12
done

I will check it in on the NSPRPUB_PRE_4_5_CLIENT_BRANCH
when the code freeze on the SeaMonkey trunk is over.
If this fix is urgent you can request drivers' approval.

Comment 8

[Niels Reedijk]

Comment on attachment 165205 [details] [diff] [review]
The patch

Even though it is not urgent, I just want this bug to be fully fixed
everywhere. This is a BeOS only change, and, as before, will not touch anything
other than beos files.

Comment 9

[Wan-Teh Chang]

I checked in the patch on the NSPRPUB_PRE_4_2_CLIENT_BRANCH
(post Mozilla 1.8 Alpha 5).