Closed Bug 462869 Opened 16 years ago Closed 16 years ago

Firefox 3 silently fails to write downloaded files

Categories

(Toolkit :: Downloads API, defect)

x86
Linux
defect
Not set
normal

Tracking

()

VERIFIED DUPLICATE of bug 429827

People

(Reporter: ofourdan, Unassigned)

Details

Attachments

(1 file)

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.3) Gecko/2008092416 Firefox/3.0.3 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.3) Gecko/2008092416 Firefox/3.0.3 When saving a file to a non user writable directory, Firefox 3 give no notification to the user but an exception error in the log: There is no notification of failure to the user but a message in the console (if FF3 was started from a terminal): *** exception in validateLeafName: [Exception... "Component returned failure code: 0x80520015 (NS_ERROR_FILE_ACCESS_DENIED) [nsIFile.create]" nsresult: "0x80520015 (NS_ERROR_FILE_ACCESS_DENIED)" location: "JS frame :: file:///usr/lib/xulrunner-1.9/components/nsHelperAppDlg.js :: anonymous :: line 271" data: no] Reproducible: Always Steps to Reproduce: 1. Download a file 2. Use "Save to disk" instead of "Open with" 3. Select a destination directory that is not writable to the user Actual Results: There is no notification of failure to the user but a message in the console (if FF3 was started from a terminal): *** exception in validateLeafName: [Exception... "Component returned failure code: 0x80520015 (NS_ERROR_FILE_ACCESS_DENIED) [nsIFile.create]" nsresult: "0x80520015 (NS_ERROR_FILE_ACCESS_DENIED)" location: "JS frame :: file:///usr/lib/xulrunner-1.9/components/nsHelperAppDlg.js :: anonymous :: line 271" data: no] Expected Results: A dialog box notifying that the file was not saved. The fix is actually fairly simple, in "nsHelperAppDlg.js", the call to aLocalFile.create() must be protected by a try/catch.
Attached patch Proposed patch to fix the issue (deleted) — Splinter Review
Status: UNCONFIRMED → RESOLVED
Closed: 16 years ago
Resolution: --- → DUPLICATE
Status: RESOLVED → UNCONFIRMED
Resolution: DUPLICATE → ---
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attachment #346056 - Flags: review?(gavin.sharp)
Component: General → Download Manager
Product: Firefox → Toolkit
QA Contact: general → download.manager
Version: unspecified → Trunk
Bug 429827 should have fixed this.
Status: NEW → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → DUPLICATE
Attachment #346056 - Flags: review?(gavin.sharp)
Thanks for the patch anyways, though, Olivier!
Verified dup
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: