Closed
Bug 1336243
Opened 7 years ago
Closed 7 years ago
Launcher after install fails to start Firefox Nightly
Categories
(Firefox :: Installer, defect)
Firefox
Installer
Tracking
()
VERIFIED
FIXED
Firefox 54
Tracking | Status | |
---|---|---|
firefox51 | --- | unaffected |
firefox52 | --- | unaffected |
firefox53 | --- | unaffected |
firefox54 | --- | verified |
People
(Reporter: jmjjeffery, Assigned: molly)
References
Details
(Keywords: regression)
Attachments
(1 file)
After using the .exe file for an install of the browser when you get to the last panel that shows 'Finish' there is a check-box pre-checked to "Launch Firefox" Nightly builds are failing to launch. It will start from the Icon on the Desktop. Mozilla/5.0 (Windows NT 10.0; WOW64; rv:54.0) Gecko/20100101 Firefox/54.0 win10 x64 cset 1d025ac534a6333a8170a59a95a8a3673d4028ee good cset ae91b2b5bb69ffd18b4679188730d0c6af3b4a95 bad regression range: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=1d025ac534a6333a8170a59a95a8a3673d4028ee&tochange=ae91b2b5bb69ffd18b4679188730d0c6af3b4a95 I guess I don't know how to test the Launch failure with Mozregressioin
Assignee | ||
Comment 1•7 years ago
|
||
Thanks for the report, and for getting the regression range. My bug 1324617 patch is the one that caused this; I broke the method used for finding the right executable to launch. It's a simple fix, so I'll have a patch up shortly.
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
Matt, this would probably be a good patch to have agashlin review. Sound good?
Flags: needinfo?(mhowell)
Assignee | ||
Comment 4•7 years ago
|
||
Totally agree, been having similar thoughts myself. Switching request.
Flags: needinfo?(mhowell)
Assignee | ||
Updated•7 years ago
|
Attachment #8833115 -
Flags: review?(robert.strong.bugs) → review?(agashlin)
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8833115 [details] Bug 1336243 - Fix a regression from bug 1324617 in launching the browser post-install. https://reviewboard.mozilla.org/r/109336/#review110482 ::: browser/installer/windows/nsis/installer.nsi:842 (Diff revision 1) > UAC::ExecCodeSegment $0 > ${EndIf} > FunctionEnd > > Function LaunchAppFromElevatedProcess > - ; Find the installation directory when launching using GetFunctionAddress > + Pop $0 It looks like when the original code was written (no later than September 2007 [1]), variables were not copied between the elevated and unelevated installers on UAC::ExecCodeSegment. History.txt [2] suggests this was added in November 2007: > v0.0.6d - 20071108 (AndersK) > +Now syncs basic registers/variables before calling UAC::*Exec* and UAC::ExecCodeSegment (r0-r9,R0-R9,$CMDLINE,$INSTDIR,$OUTDIR,$EXEDIR,$LANGUAGE) Since we do have that support now [3], wouldn't it make sense to use $INSTDIR directly instead of the UAC::StackPush and Pop? I did a quick test and it seems to work as expected. [1] https://hg.mozilla.org/mozilla-central/rev/3305baa9f049 [2] http://nsis.sourceforge.net/mediawiki/images/8/8f/UAC.zip [3] The current UAC.dll is from bug 571387, in 2010
Comment 6•7 years ago
|
||
This is my first review request, I think may have gone too deep into it. If you don't think this is worth changing I'll drop that issue.
Updated•7 years ago
|
status-firefox51:
--- → unaffected
status-firefox52:
--- → unaffected
status-firefox53:
--- → unaffected
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8833115 -
Flags: review?(robert.strong.bugs)
Comment 8•7 years ago
|
||
mozreview-review |
Comment on attachment 8833115 [details] Bug 1336243 - Fix a regression from bug 1324617 in launching the browser post-install. https://reviewboard.mozilla.org/r/109336/#review110732
Attachment #8833115 -
Flags: review+
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e6d624b8326b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Reporter | ||
Comment 11•7 years ago
|
||
Verified FIXED today's Nightly cset: https://hg.mozilla.org/mozilla-central/rev/3e555770a90a41e04bbb4ac41b65fa2f1db6977d Win10 x64 - win32 builds.
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•