Closed Bug 1519733 Opened 1 year ago Closed 1 year ago

stop calling umask in nsOSHelperAppService

Categories

(Firefox :: File Handling, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 66
Tracking Status
firefox66 --- fixed

People

(Reporter: heycam, Assigned: heycam)

References

Details

Attachments

(1 file)

Bug 878144 removed the use of nsOSHelperAppService::mPermissions, so we can delete that field and the calls to umask in the constructor. (The umask calls are bumping into the Linux sandbox with a patch I'm writing that happens to make the nsOSHelperAppService get lazily instantiated later than it does now.)

And remove the mPermissions field, which became unused after bug 878144.
Blocks: 1519737
Assignee: nobody → cam
Status: NEW → ASSIGNED
Pushed by cmccormack@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/557a23aec3af
Stop calling umask in nsOSHelperAppService. r=paolo
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
You need to log in before you can comment on or make changes to this bug.