Closed Bug 542738 Opened 15 years ago Closed 15 years ago

nsLocalFile::IsWritable/IsReadable/IsExecutable is wrong on Solaris

Categories

(Core :: XPCOM, defect)

All
OpenSolaris
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: ginnchen+exoracle, Assigned: ginnchen+exoracle)

References

Details

Attachments

(1 file)

It is also wrong on BeOS. But I'll only fix it for Solaris here. The behavior is changed in Bug 351950, it explains why using access() is not good on Solaris. The comment said Bug 351595, it is a typo.
(In reply to comment #0) > It is also wrong on BeOS. > But I'll only fix it for Solaris here. > > The behavior is changed in Bug 351950, it explains why using access() is not > good on Solaris. > The comment said Bug 351595, it is a typo. Thanks. Now I understand what the issue was, I think. I guess we need to fix the typo in the said also! TIA
Attached patch patch (deleted) — Splinter Review
The BeOS implementation is not correct, it is better than nothing, since access() is always 0 on BeOS according to Bug 169506. On Solaris, the only problem we want to resolve if IsExecutable() with root user. Using BeOS implementation is actually a step backwards.
Assignee: nobody → ginn.chen
Status: NEW → ASSIGNED
Attachment #423978 - Flags: review?(mozbugz)
Comment on attachment 423978 [details] [diff] [review] patch This makes solaris work more like linux with the exception of IsExecutable(). That sounds fine to me. Please make sure that you run the tests and nothing is an issue.
Attachment #423978 - Flags: review?(mozbugz) → review+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: