Closed
Bug 747921
Opened 12 years ago
Closed 12 years ago
Intermittent failures on app installation on windows 7
Categories
(Firefox Graveyard :: Web Apps, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 15
People
(Reporter: anant, Assigned: Felipe)
References
Details
(Whiteboard: [marketplace-beta+])
Attachments
(1 file)
929 bytes,
patch
|
TimAbraldes
:
review+
|
Details | Diff | Splinter Review |
Timestamp: 4/23/2012 8:09:57 AM Error: Error installing app: [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIWindowsRegKey.open]" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: resource:///modules/WebappsInstaller.jsm :: <TOP_LEVEL> :: line 224" data: no] Source File: resource:///modules/WebappsInstaller.jsm Line: 36
Comment 1•12 years ago
|
||
Which app(s) did you try to install? And from which website (f.e. marketplace-dev.allizom.org)? And while running which nightly build? A bit more information would be most helpful. ;-)
OS: Mac OS X → Windows 7
Comment 2•12 years ago
|
||
Not getting a reproduction of this bug on Win 7 64-bit with the current nightly. Anant could you clarify the following things: - What build are you running (e.g. the latest nightly) - If the problem still reproduces with the current nightly - Reproduction steps
Reporter | ||
Comment 3•12 years ago
|
||
Sorry, I mostly just wanted to log the error message so we don't forget. The problem occurred on Todd's Windows Laptop this morning, and he will followup with information on his setup shortly.
Windows: Win 7, version 6.1 Nightly: 14.0a1 (2012-04-23) https://marketplace.mozilla.org/en-US/ Tried: Lord of Ultima, Boston Globe, GodFather, Lanyard, etc.... Reproduce: click install on any app. Gives error message "App Installation not allowed"
Reporter | ||
Comment 5•12 years ago
|
||
STR: The error occurs when installing an app from anywhere (I just tried installing one from the mozillalabs appdir). It's the latest nightly from today. I have a hunch that this has something to do with permissions (the current user being unable to write to the registry), or that the registry is just corrupt.
Comment 6•12 years ago
|
||
(In reply to Anant Narayanan [:anant] from comment #5) > I have a hunch that this has something to do with permissions (the current > user being unable to write to the registry), or that the registry is just > corrupt. uninstallKey.open(uninstallKey.ROOT_KEY_CURRENT_USER, "SOFTWARE\\Microsoft\\Windows\\" + "CurrentVersion\\Uninstall", uninstallKey.ACCESS_WRITE); That's the line of code where the failure is occurring. Does not look like it happens on every machine, only certain Windows machines. I agree with Anant - Sounds like this might be a permissions issue. Tim - Any ideas?
Updated•12 years ago
|
Summary: App installation fails on Windows 7 → Intermittent failures on app installation on windows 7
Reporter | ||
Comment 7•12 years ago
|
||
Looking at the code a bit more, it looks like we're checking the registry to see if the app was previously installed and if so, we remove it. I think failure to do so shouldn't raise an error in the install process, or maybe we can punt on it until later on when we try to add a registry key? Anyway, that's just a band-aid, we still have to figure out the core problem of why that Component is failing on that particular Windows setup.
Updated•12 years ago
|
Whiteboard: [marketplace-beta]
Updated•12 years ago
|
Whiteboard: [marketplace-beta] → [marketplace-beta+]
Comment 8•12 years ago
|
||
I just tested this on a Win 7 machine and got an App manifest is invalid error.
Comment 9•12 years ago
|
||
This is what I see in the error console when I tried to install the Evernote app: marketplace-dev.allizom.org : server does not support RFC 5746, see CVE-2009-3555 stage.evernote.com : server does not support RFC 5746, see CVE-2009-3555
Comment 10•12 years ago
|
||
According to SUMO, it appears this may be related to some issue with Java? Really? http://support.mozilla.org/en-US/questions/746438
Comment 11•12 years ago
|
||
I was able to successfully install Sandglaz, but Evernote and Boston Globe failed.
Reporter | ||
Comment 12•12 years ago
|
||
The CVE messages are just warnings, they are not the cause of the problem.
Comment 13•12 years ago
|
||
(In reply to Ragavan S [:rags] from comment #11) > I was able to successfully install Sandglaz, but Evernote and Boston Globe > failed. Sounds like what you are describing an existing marketplace dev bug - bug 747252.
Comment 14•12 years ago
|
||
Hmm. I got an idea for testing that might reveal the root cause of this issue - We had a problem in the past with the extension involving "fresh" builds of windows 7 I think that involved registry keys (bug 723355). As a result, the application could not be installed on the user's machine. However, when they installed Chrome on their machine, a certain registry key was written, which as a result, lead to allowing app installations. First, do we know if logic behind bug 723355 was taken into account in the new implementation? Checking the diff, I wonder if this is correct. Myk or Tim - Could you take a look at the patch on that bug and comment here on your opinion? I'll see if I get a fresh build of Windows 7 to test this. Todd - Could you give me more details on the machine information - Things like how long you've had the windows 7 OS installed, how old the machine is, whether you've installed Chrome on that machine or not, etc.
Comment 15•12 years ago
|
||
Followup questions to comment 14: Myk or Tim - If the registry key in comment 6 doesn't exist, do we ensure we create it to avoid erroring out? Todd - We might want to look at the windows registry on your machine.
Updated•12 years ago
|
Assignee: nobody → felipc
Assignee | ||
Comment 16•12 years ago
|
||
Straightforward. The code that _creates_ the key had no problem dealing with the nonexistent key, it's only the code that tries to clean-up a previous installation first that wasn't coping well.
Attachment #618366 -
Flags: review?(tabraldes)
Comment 17•12 years ago
|
||
Comment on attachment 618366 [details] [diff] [review] Patch Review of attachment 618366 [details] [diff] [review]: ----------------------------------------------------------------- Tested locally and chatted with Felipe about the patch. Looks good!
Attachment #618366 -
Flags: review?(tabraldes) → review+
Assignee | ||
Comment 18•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2b118a11328b
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Comment 20•12 years ago
|
||
Verified through Todd today in the apps show and tell.
Status: RESOLVED → VERIFIED
Updated•12 years ago
|
Flags: in-moztrap-
Updated•12 years ago
|
QA Contact: jsmith
Updated•8 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•