Closed Bug 1037649 Opened 10 years ago Closed 10 years ago

Failed to remove file extensions\trash\firefox-hotfix@mozilla.org.xpi: [Exception... "Component returned failure code: 0x80520015 (NS_ERROR_FILE_ACCESS_DENIED) [nsIFile.remove]

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: gps, Assigned: gps)

References

Details

Report from Catalin in Firefox 10 through 17.

Error: ERROR addons.xpi: Failed to remove file C:\Users\camelia.badau\AppData\Roaming\Mozilla\Firefox\Profiles\2bj5ep00.21\extensions\trash\firefox-hotfix@mozilla.org.xpi: [Exception... "Component returned failure code: 0x80520015 (NS_ERROR_FILE_ACCESS_DENIED) [nsIFile.remove]"  nsresult: "0x80520015 (NS_ERROR_FILE_ACCESS_DENIED)"  location: "JS frame :: resource:///modules/XPIProvider.jsm :: recursiveRemove :: line 1256"  data: no]

Beside this error a Mozilla\Firefox\Profiles\1modinl0.9\extensions\trash\firefox-hotfix@mozilla.org.xpi files remains undeleted in the Profile folder.
Please note that the update finishes with success and this issue doesn't affect the update.

---

My initial reaction is "meh, this is something weird in AddonManager land." But, I'd like to investigate first to make sure it really is benign.
I am unable to reproduce this.

Steps attempted:

1) Install Firefox 11 on Windows 7
2) Hack up prefs to use dev update server and enable update and extension logging
3) Restart
4) Opt in to Telemetry via toolbar
5) Trigger 3 manual update checks via privileged console
6) Accept UAC prompt after hotfix is downloaded
7) Restart
8) Open console and don't see this error
Flags: needinfo?(catalin.varga)
Perhaps Blair can shed some light on this trash directory, what might be happening, and whether it is worth digging into.
Flags: needinfo?(bmcbride)
This is safe, and in certain conditions is even expected due to file locks on Windows. The directory should be emptied the next time Firefox starts up. 

Context: The trash directory exists partly due to this type of problem, and partly so that we can do a rollback when something like an upgrade fails.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: needinfo?(bmcbride)
Resolution: --- → INVALID
Flags: needinfo?(catalin.varga)
You need to log in before you can comment on or make changes to this bug.