26859 - nsLocalFile::Exists not consistent across platforms

Status: VERIFIED INVALID

(Core :: XPCOM, defect, P3)

Description

[John Bandhauer]

nsLocalFile::Exists does wildly different things on different platforms with 
files that do not exist.

The Unix version returns an error code. It *looks* like the Win and Mac code 
will return a 'false' out param as expected.

Are there tests (especially platform parity tests) for this subsystem? This is 
exacly the kind of thing that you could test using JavaScript.

What is the level of consistency of the other methods to these classes?

Comment 1

[Doug Turner (:dougt)]

chris, can you take a look at this.  

jband, for javascript tests, look at:
http://lxr.mozilla.org/seamonkey/source/xpcom/tests/utils/

Comment 2

[Christopher Blizzard (:blizzard)]

Wasn't this fixed a while ago?

Comment 3

[Pete Collins (MDG)]

If you check for a file on unix and it doesn't exist, it throws an exception.

I listed a whole bunch of bugs i found from js in the xpcom ng.

Like append not working, read file not working etc.

pete

Comment 4

[Pete Collins (MDG)]

$ cat /home/petejc/IO/foo299.dat 
This is line R 
This is line R 
This is line R 

fine creates this file no probs accept for the permissions: 

$ ls -l /home/petejc/IO/foo299.dat 
-rwxr-xr-x  1 petejc  petejc  45 Feb  4 10:37 /home/petejc/IO/foo299.dat 

this is 755 permissions not 777 . .  . 

the file is being created with this call:  file.create(0,0777); 

now when i try to append to this file this is what happens: 

^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@^@This
is line S 
This is line S 
This is line S 

It seems to be converting the old data to garbage. If you count the garbage
characters i beleive they are the correct number of characters from the
original text. 
  

Comment 5

[Pete Collins (MDG)]

Attachment: javascript test file:

Comment 6

[Christopher Blizzard (:blizzard)]

Accepting the bug for more inspection and from what it sounds like, fixing.

Comment 7

[Christopher Blizzard (:blizzard)]

I've verified that nsIFile::Exists() works on Unix and doesn't throw any
exceptions.  I tested it by touching and deleting the file and trying
file.exists() from xpcshell and it worked as expected.

Comment 8

[leger]

Marking Verified per blizzard's last comment.

Comment 9

[Christopher Blizzard (:blizzard)]

Please ignore the spam.  Changing address.

Comment 10

[Christopher Blizzard (:blizzard)]

bustage from my reassign

Comment 11

[Christopher Blizzard (:blizzard)]

bustage from my reassign