108276 - The file socket to mailbox can be opened in PR_RDONLY

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect)

Description

[Navin Gupta]

Right now we open local folders/mailbox in PR_RDWR mode. I have checked the code
and we don't need to open in PR_RDWR. We can open in PR_RDONLY because all we 
do through these file connections is read. Whenever we do a write we open
outputfileStream separately. This should help in reducing closing time PR_Close 
where it slow. Also it will help in safeguarding unwanted writes.

Comment 1

[Navin Gupta]

Attachment: fix

Comment 2

[Navin Gupta]

All mailbox actions have been tested and they work fine. compacting also tested. 

cc bienvenu for review.

Comment 3

[David :Bienvenu]

well, this looks ok, but ideally we would open the file rdrw if we were going to
change the read flag, for example, and then pass the open file handle to the
code that updates the folder flag. So we may have to revisit this.

Comment 4

[David :Bienvenu]

Comment on attachment 56336 [details] [diff] [review]
fix

r=bienvenu, but see my comments.

Comment 5

[Scott MacGregor]

Comment on attachment 56336 [details] [diff] [review]
fix

sr=mscott

Can you make sure things like compact folders still work after this change? Just making sure they (like other write operations) are going through a different code path.

Comment 6

[Navin Gupta]

this was checked in long time back. 

Comment 7

[[e-mail] (gone - use [e-mail] instead)]

Navin, safe to verify this using LXR?

Comment 8

[Navin Gupta]

yep, it is. it sure has been tested on trunk for over a month now...

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

Verified FIXED via code inspection via LXR:

1.93 naving%netscape.com Nov 5 20:25 108276. Open the mailbox in PR_RDONLY mode
because only read operations are done using
this connection. should cut down closing mailbox (PR_Close()) time.