Closed
Bug 1120673
Opened 9 years ago
Closed 9 years ago
Fx 35 installer crashes on XP x86 SP3 at the end (creating shortcuts) if the xp firewall service is stopped
Categories
(Firefox :: Installer, defect)
Tracking
()
VERIFIED
FIXED
Firefox 38
People
(Reporter: al_9x, Assigned: robert.strong.bugs)
References
Details
(Keywords: regression)
Attachments
(3 files, 1 obsolete file)
7.97 KB,
patch
|
bbondy
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
Sylvestre
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
37.42 KB,
image/png
|
Details | |
36.82 KB,
image/png
|
Details |
reproducible in a vm AppName: setup.exe AppVer: 1.0.0.0 ModName: msvcrt.dll ModVer: 7.0.2600.5512 Offset: 00032a16
Assignee | ||
Comment 1•9 years ago
|
||
Unable to reproduce with WinXP SP2 on a VM (had WinXP SP 2 handy). Kamil, can you try to reproduce? Thanks!
Flags: needinfo?(kjozwiak)
Assignee | ||
Comment 2•9 years ago
|
||
Also tried installing on top of an existing install and was unable to reproduce on WinXP SP2
Comment 3•9 years ago
|
||
Used the following build: - http://ftp.mozilla.org/pub/mozilla.org/firefox/candidates/35.0-candidates/build3/ Checked on the following OS's: - Win XP Pro x86 SP3 - PASSED (ran through it two times) - Win XP Pro x86 SP2 - PASSED (ran through it two times) - Win XP Pro x64 SP2 - PASSED (ran through it two times) Looks like it's working for me. I also launched fx a few times via the shortcut and everything worked without any issues. Couldn't reproduce any crashes :(
Flags: needinfo?(kjozwiak)
Assignee | ||
Comment 4•9 years ago
|
||
Reporter, does this reproduce everytime on your system?
Flags: needinfo?(al_9x)
to repro, stop the windows firewall service _________________________________ call stack from drwtsn32.log msvcrt!wscanf+0x750 msvcrt!printf+0x35 liteFirewallW+0x1292 liteFirewallW!AddRule+0x4c setup+0x21d8 setup+0x13ba setup+0x13ba __________________________________ setup should not crash if firewall is off, but why are you adding incoming firewall rules in the first place? it certainly is not needed fro browsing.
Flags: needinfo?(al_9x)
Summary: Fx 35 installer crashes on XP x86 SP3 at the end (creating shortcuts) → Fx 35 installer crashes on XP x86 SP3 at the end (creating shortcuts) if the xp firewall service is stopped
Assignee | ||
Comment 6•9 years ago
|
||
Thanks!
Assignee: nobody → robert.strong.bugs
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
you didn't answer: why are you adding incoming firewall rules in the first place? it certainly is not needed for browsing.
Comment 8•9 years ago
|
||
Added STR (reproduced twice): 1) Download http://ftp.mozilla.org/pub/mozilla.org/firefox/candidates/35.0-candidates/build3/ 2) Disable "Windows Firewall/Internet Connection Sharing (ICS)" under Services 3) Install the above build, will receive the crash during the "Create shortcut..." process
Assignee | ||
Comment 9•9 years ago
|
||
Sorry, didn't see the question. It was added to support bug 1054959. To disable it you can change the rule to a deny rule.
Comment 10•9 years ago
|
||
Robert, here's more information if you need it: Turning off "Windows Firewall/Internet Connection Sharing (ICS)" - Win XP SP2 x86 - Reproduced - Win XP SP3 x86 - Reproduced - Win XP SP2 x64 - Reproduced Turning off "Windows Firewall" - Win 7 Pro SP1 x86 - Couldn't reproduce the crash with the service mentioned above turned off (x2) - Win 7 Pro SP1 x64 - Couldn't reproduce the crash with the service mentioned above turned off (x2) - Win 8 x86 - Couldn't reproduce the crash with the service mentioned above turned off (x2) - Win 8.1 x86 - Couldn't reproduce the crash with the service mentioned above turned off (x2) - Win 8 x64 - Couldn't reproduce the crash with the service mentioned above turned off (x2) - Win 8.1 x64 - Couldn't reproduce the crash with the service mentioned above turned off (x2) I'll check Vista once I have them installed tomorrow.. Haven't added Vista since upgrading to VM Fusion 7. They also take the longest to install :/
Assignee | ||
Comment 11•9 years ago
|
||
Thanks Kamil! I have Vista VMs so no worries.
Assignee | ||
Comment 12•9 years ago
|
||
Attachment #8548638 -
Flags: review?(netzen)
Assignee | ||
Comment 13•9 years ago
|
||
[Tracking Requested - why for this release]:
status-firefox35:
--- → affected
status-firefox36:
--- → affected
status-firefox37:
--- → affected
status-firefox38:
--- → affected
status-firefox-esr31:
--- → unaffected
Updated•9 years ago
|
Attachment #8548638 -
Flags: review?(netzen) → review+
Assignee | ||
Comment 14•9 years ago
|
||
Attachment #8548638 -
Attachment is obsolete: true
Assignee | ||
Comment 15•9 years ago
|
||
Comment on attachment 8549368 [details] [diff] [review] patch with comments and register preservation Tested successfully on Win XP, Win Vista, Win 7, and Win 8.1 with and without the service and with an admin and non-admin accounts.
Attachment #8549368 -
Flags: review+
Assignee | ||
Comment 16•9 years ago
|
||
Comment on attachment 8549368 [details] [diff] [review] patch with comments and register preservation Brian, I wanted to run the register preservation changes by you. Thanks!
Attachment #8549368 -
Flags: review?(netzen)
Assignee | ||
Comment 17•9 years ago
|
||
[Tracking Requested - why for this release]: On Win XP when the Firewall service isn't running the installer crashes towards the end after creating shortcuts. If there is a Firefox 35 point release I'd like to get this in.
tracking-firefox35:
--- → ?
Assignee | ||
Updated•9 years ago
|
Attachment #8549368 -
Flags: review+ → review?(spohl.mozilla.bugs)
Comment 18•9 years ago
|
||
Comment on attachment 8549368 [details] [diff] [review] patch with comments and register preservation Review of attachment 8549368 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me Robert! One nit inline. ::: browser/installer/windows/nsis/shared.nsh @@ +1570,5 @@ > + Push $R7 > + Push $R6 > + Push "false" > + > + System::Call 'advapi32::OpenSCManagerW(n, n, i ${SC_MANAGER_ALL_ACCESS}) i.R6' I'd ask for less access here if possible. Could fail for some account setups.
Attachment #8549368 -
Flags: review?(netzen) → review+
Comment 19•9 years ago
|
||
Hey Robert, I'm awfully unfamiliar with nsis. Did you want me to review everything, or was there something specific that you wanted my feedback on? I'm happy to go over the patch, with the caveat that it may take me some time to understand it well enough to give it my r+. If this is urgent I'm happy to defer to Brian's review. Just let me know. Thanks!
Flags: needinfo?(robert.strong.bugs)
Comment 20•9 years ago
|
||
We're still collecting potential drivers & ride along issues, tracking this as a potential ride along.
Assignee | ||
Updated•9 years ago
|
Attachment #8549368 -
Flags: review?(spohl.mozilla.bugs)
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(robert.strong.bugs)
Comment 21•9 years ago
|
||
Since jsavage (who normally writes english articles) is on PTO, ideato, madalina and I wrote a SUMO KB article about this: https://support.mozilla.org/en-US/kb/firefox-installer-crashes-windows-xp
Assignee | ||
Comment 22•9 years ago
|
||
Pushed to mozilla-central with sheriff approval https://hg.mozilla.org/mozilla-central/rev/89a190592267
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 38
Assignee | ||
Updated•9 years ago
|
Blocks: 1090535
Keywords: regression
Assignee | ||
Comment 23•9 years ago
|
||
Comment on attachment 8549368 [details] [diff] [review] patch with comments and register preservation Approval Request Comment [Feature/regressing bug #]: bug 1090535 [User impact if declined]: Installing on Win XP when the Firewall service is not running will crash the installer [Describe test coverage new/current, TBPL]: Manually tested on Windows with and without the Firewall service running. [Risks and why]: minimal due to lots of manual testing. [String/UUID change made/needed]: None. Note: I'd also like to get this into release if there is a point release
Attachment #8549368 -
Flags: approval-mozilla-beta?
Attachment #8549368 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
Attachment #8549368 -
Flags: approval-mozilla-beta?
Attachment #8549368 -
Flags: approval-mozilla-beta+
Attachment #8549368 -
Flags: approval-mozilla-aurora?
Attachment #8549368 -
Flags: approval-mozilla-aurora+
Comment 24•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/374a9adf2786 https://hg.mozilla.org/releases/mozilla-beta/rev/0f43b4df53bb
Assignee | ||
Comment 25•9 years ago
|
||
Comment on attachment 8549368 [details] [diff] [review] patch with comments and register preservation Approval Request Comment [Feature/regressing bug #]: bug 1090535 [User impact if declined]: Installing on Win XP when the Firewall service is not running will crash the installer [Describe test coverage new/current, TBPL]: Manually tested on Windows with and without the Firewall service running. [Risks and why]: minimal due to lots of manual testing. [String/UUID change made/needed]: None.
Attachment #8549368 -
Flags: approval-mozilla-release?
Updated•9 years ago
|
Flags: qe-verify+
Comment 26•9 years ago
|
||
Test Cases used for verification: Test Case #1: * Disabled "Windows Firewall/Internet Connection Sharing (ICS)" * Installed build: http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2015-01-21-03-02-03-mozilla-central/ * re-launched fx several times and ensured there wasn't any crashes * re-enabled "Windows Firewall/Internet Connection Sharing (ICS)" and launched fx several times ** received the Firewall warning and selected "unblock" ** ensured that Nightly appeared under "Exceptions" list under "Windows Firewall" * Disabled "Windows Firewall/Internet Connection Sharing (ICS)" once again and opened fx several times * re-enabled "Windows Firewall/Internet Connection Sharing (ICS)" and launched fx several times Test Case #2: * installed an older version of Nightly * once installed, disabled "Windows Firewall/Internet Connection Sharing (ICS)" * proceeded with Nightly update with the firewall disabled * ensured that Nightly updates without any issues or crashes and fx launches without any problems * enabled "Windows Firewall/Internet Connection Sharing (ICS)" and launched FX several times Went through the above test cases on the following OS's: - Windows XP Pro x86 SP2 - PASSED - Windows XP Pro x86 SP3 - PASSED - Windows XP Pro x64 SP2 - PASSED One thing that I noticed while going through this: * Nightly appears as "Nightly (C:\Program Files\Nightly)" under the exception list when installing fx while the firewall service is enabled ** Check installWithFirewallEnabled image * Nightly appears as "Nightly" under the exception list when installing fx while the firewall is enabled via the "Unblock" prompt ** Check Robert, this doesn't seem like a problem but just making sure this doesn't lead to anything more serious?
Flags: needinfo?(robert.strong.bugs)
Comment 27•9 years ago
|
||
Assignee | ||
Comment 28•9 years ago
|
||
(In reply to Kamil Jozwiak [:kjozwiak] from comment #26) > Created attachment 8552575 [details] > installWithFirewallEnabled.png > > Test Cases used for verification: > > Test Case #1: > > * Disabled "Windows Firewall/Internet Connection Sharing (ICS)" > * Installed build: > http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2015-01-21-03-02-03- > mozilla-central/ > * re-launched fx several times and ensured there wasn't any crashes > * re-enabled "Windows Firewall/Internet Connection Sharing (ICS)" and > launched fx several times > ** received the Firewall warning and selected "unblock" > ** ensured that Nightly appeared under "Exceptions" list under "Windows > Firewall" > * Disabled "Windows Firewall/Internet Connection Sharing (ICS)" once again > and opened fx several times > * re-enabled "Windows Firewall/Internet Connection Sharing (ICS)" and > launched fx several times > > Test Case #2: > > * installed an older version of Nightly > * once installed, disabled "Windows Firewall/Internet Connection Sharing > (ICS)" > * proceeded with Nightly update with the firewall disabled > * ensured that Nightly updates without any issues or crashes and fx launches > without any problems > * enabled "Windows Firewall/Internet Connection Sharing (ICS)" and launched > FX several times > > Went through the above test cases on the following OS's: > > - Windows XP Pro x86 SP2 - PASSED > - Windows XP Pro x86 SP3 - PASSED > - Windows XP Pro x64 SP2 - PASSED > > One thing that I noticed while going through this: > > * Nightly appears as "Nightly (C:\Program Files\Nightly)" under the > exception list when installing fx while the firewall service is enabled > ** Check installWithFirewallEnabled image > * Nightly appears as "Nightly" under the exception list when installing fx > while the firewall is enabled via the "Unblock" prompt > ** Check > > Robert, this doesn't seem like a problem but just making sure this doesn't > lead to anything more serious? That's intentional so we are able to distinguish between user created exceptions and exceptions that we create. Thanks again!
Flags: needinfo?(robert.strong.bugs)
Comment 29•9 years ago
|
||
Awesome, thanks Robert!
Comment 30•9 years ago
|
||
Used the following build with the steps from comment #26: * http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2015-01-18-00-40-06-mozilla-aurora/ * http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2015-01-21-00-40-11-mozilla-aurora/ - Ensured that both test cases from comment #26 worked without any issues - Ensured that aurora appeared as "FirefoxDeveloperEdition" under the exception list of Windows Firewall when manually added - Ensured taht aurora appeared as "FirefoxDeveloperEdition (C:\Program Files\Firefox Developer Edition)" under the exception list of Windows Firewall when automatically added OS's: - Windows XP Pro x86 SP2 - PASSED - Windows XP Pro x86 SP3 - PASSED - Windows XP Pro x64 SP2 - PASSED
Comment 31•9 years ago
|
||
Used the following build with the steps from comment #26: * http://ftp.mozilla.org/pub/mozilla.org/firefox/candidates/36.0b1-candidates/build2/win32/en-US/ * http://ftp.mozilla.org/pub/mozilla.org/firefox/candidates/36.0b2-candidates/build1/win32/en-US/ - Ensured that both test cases from comment #26 worked without any issues - Ensured that beta appeared as "Firefox" under the exception list of Windows Firewall when manually added - Ensured that beta appeared as "Firefox (C:\Program Files\Mozilla Firefox)" under the exception list of Windows Firewall when automatically added OS's: - Windows XP Pro x86 SP2 - PASSED - Windows XP Pro x86 SP3 - PASSED - Windows XP Pro x64 SP2 - PASSED
Updated•9 years ago
|
Attachment #8549368 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Comment 33•9 years ago
|
||
Went through verification using the following build: - http://ftp.mozilla.org/pub/mozilla.org/firefox/candidates/35.0.1-candidates/build1/win32/en-US/ - ensured that the fx firewall entry is being added into the exceptions list when installing - launched fx several times and ensured that it opens without any crashes - ensured that fx installs without any issues/crashes when the firewall service has been disabled - launched fx several times and ensured that it opens when the firewall service has been disabled One thing I noticed is that users don't get the firewall warning if they re-enable the firewall service and launch FX (if it hasn't already been added into the exceptions). Talked to Robert and it appears like it's MS code that decides when to prompt and when not to prompt users. Os's used: - Windows XP Pro x86 SP2 - PASSED - Windows XP Pro x86 SP3 - PASSED - Windows XP Pro x64 SP2 - PASSED
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•