437216 - Add support of NTFS permissions in nsLocalFile::IsWritable

Status: NEW

(Core :: XPCOM, enhancement)

Description

[Ronny Perinke]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.8.1.15pre) Gecko/20080419 Firefox/2.0.0.15pre (Sephiroth/SSE2)
Build Identifier: 

IsWritable only checks whether or not the read-only attribute is set. But on NTFS, there could be the situation where it's not set and the user has no right to write to a file or create files in the directory.

I wrote a little example how to retrieve the effective rights for the user that runs the application (owner of the calling thread or process).

Reproducible: Always

Comment 1

[Ronny Perinke]

Attachment: example how to retrieve the effective rights

Comment 2

[Dave Townsend [:mossop]]

Jim, is this something you might be able to take? We currently have all sorts of nasty hacks in the extension manager to verify that we can add files to a directory because this API doesn't work correctly at the moment.

Comment 3

[Jim Mathies [:jimm]]

Finally getting back to this.

Comment 4

[Jim Mathies [:jimm]]

Attachment: patch

Comment 5

[Jim Mathies [:jimm]]

Attachment: patch

some last minute cleanup totally broke that last patch. pushed to try, I might try to work up a test as well. since we don't use ntfs on our test slaves it wouldn't run normally but devs would still be able to test.

Comment 6

[Sylvestre Ledru [:Sylvestre]]

No assignee, updating the status.

Comment 7

[Sylvestre Ledru [:Sylvestre]]

No assignee, updating the status.

Comment 8

[Sylvestre Ledru [:Sylvestre]]

No assignee, updating the status.

Comment 9

[Sylvestre Ledru [:Sylvestre]]

No assignee, updating the status.