Closed
Bug 966626
Opened 11 years ago
Closed 11 years ago
CommandExecuteHandler.exe - Application Error
Categories
(Firefox :: Shell Integration, defect)
Tracking
()
VERIFIED
FIXED
Firefox 29
People
(Reporter: streetwolf52, Assigned: jimm)
References
Details
(Keywords: regression, Whiteboard: [metro] [beta28] p=1)
Attachments
(3 files)
User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release)
Build ID: 20140131171045
Steps to reproduce:
Open up Fx29 by clicking on an external link from another program.
Actual results:
Event Log:
Faulting application name: CommandExecuteHandler.exe, version: 29.0.0.5142, time stamp: 0x52eab06b
Faulting module name: SHELL32.dll, version: 6.3.9600.16474, time stamp: 0x52902a5b
Exception code: 0xc0000005
Fault offset: 0x001c6322
Faulting process id: 0x10d8
Faulting application start time: 0x01cf1ed7b44a29e6
Faulting application path: C:\Users\Gary\My Programs\firefox\CommandExecuteHandler.exe
Faulting module path: C:\Windows\SYSTEM32\SHELL32.dll
Report Id: f28bde87-8aca-11e3-83f3-c86000a0a026
Faulting package full name:
Faulting package-relative application ID:
Expected results:
No error when Fx29 is opened via an external link.
Reporter | ||
Updated•11 years ago
|
Component: Untriaged → General
Keywords: regression
Reporter | ||
Comment 1•11 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/c0ee7e389205 - Good
http://hg.mozilla.org/integration/mozilla-inbound/rev/a41fdfce8810 - Bad
Even though I get this error and the one that Nightly needs to be closed replying yes keeps me in Nightly with no Nightly crash.
Assignee | ||
Comment 2•11 years ago
|
||
A few question -
1) do you have the inbound build set as the default system browser?
2) if it's the default, which front end do you have selected? (desktop/metro)
3) Do you have other revs of firefox installed on the system?
Would you mind posting the log of the ceh doing its thing on this system?
https://wiki.mozilla.org/Firefox/Windows_8_Integration#Diagnosing_Command_Execute_Handler_Issues
Assignee | ||
Updated•11 years ago
|
Whiteboard: [metro] [triage]
Assignee | ||
Updated•11 years ago
|
Component: General → Shell Integration
Version: 29 Branch → 28 Branch
Reporter | ||
Comment 3•11 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #2)
> A few question -
>
> 1) do you have the inbound build set as the default system browser?
> 2) if it's the default, which front end do you have selected? (desktop/metro)
> 3) Do you have other revs of firefox installed on the system?
>
> Would you mind posting the log of the ceh doing its thing on this system?
>
> https://wiki.mozilla.org/Firefox/
> Windows_8_Integration#Diagnosing_Command_Execute_Handler_Issues
1. I use inbound as my default browser.
2. Front end is desktop
3. No other revs on my system.
Would you mind posting the log of the ceh doing its thing on this system? What and where is this? Remember that Fx doesn't actually crash AFAICT.
Reporter | ||
Comment 4•11 years ago
|
||
Here's the log:
[1976] SHIMVIEW: ShimInfo(Complete)
[4076] SetSelection target: http://www.santanderbank.com/
[4076]
[4076] Initialize(open)
[4076]
[4076] IExecuteCommandApplicationHostEnvironment::GetValue()
[4076]
[4076] Previous AHE: 0
[4076]
[4076] Execute()
[4076]
[4076] Previous AHE: 0
[4076]
[4076] Desktop Launch: verb:'open' exe:'C:\Users\Gary\My Programs\firefox\firefox.exe' params:'-url "http://www.santanderbank.com/"'
[4076]
[4076] Desktop browser process id: 3820
[4076]
Assignee | ||
Comment 5•11 years ago
|
||
(In reply to Gary [:streetwolf] from comment #4)
> Here's the log:
>
> [1976] SHIMVIEW: ShimInfo(Complete)
> [4076] SetSelection target: http://www.santanderbank.com/
> [4076]
> [4076] Initialize(open)
> [4076]
> [4076] IExecuteCommandApplicationHostEnvironment::GetValue()
> [4076]
> [4076] Previous AHE: 0
> [4076]
> [4076] Execute()
> [4076]
> [4076] Previous AHE: 0
> [4076]
> [4076] Desktop Launch: verb:'open' exe:'C:\Users\Gary\My
> Programs\firefox\firefox.exe' params:'-url "http://www.santanderbank.com/"'
> [4076]
> [4076] Desktop browser process id: 3820
> [4076]
After which you get a dialog about the crash?
Assignee | ||
Comment 6•11 years ago
|
||
Ok, I've managed to reproduce the event in the event viewer.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → jmathies
Reporter | ||
Comment 7•11 years ago
|
||
Reporter | ||
Comment 8•11 years ago
|
||
Reporter | ||
Comment 9•11 years ago
|
||
Pressing OK for the first error message, then close for the second one doesn't end Fx. It continues as if nothing happened.
Assignee | ||
Comment 10•11 years ago
|
||
Quicker shutdown of the ceh seems to have triggered this. When the dtor of CExecuteCommandVerb gets call, the shell item array has already been destroyed and the Release call exception faults. Releasing before the ceh ref count drops to zero solves the issue.
Attachment #8369073 -
Flags: review?(netzen)
Assignee | ||
Updated•11 years ago
|
Whiteboard: [metro] [triage] → [metro] [beta28]
Assignee | ||
Updated•11 years ago
|
Whiteboard: [metro] [beta28] → [metro] [beta28] p=1
Updated•11 years ago
|
Attachment #8369073 -
Flags: review?(netzen) → review+
Assignee | ||
Comment 11•11 years ago
|
||
Thanks for the quick turn around.
Note, I've cancelled the pending 02-01 Windows Nightly so our nightly users don't run into this. I'll land this fix directly on mc.
Assignee | ||
Comment 12•11 years ago
|
||
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•11 years ago
|
||
BTW - thanks for testing Gary!
Reporter | ||
Comment 14•11 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #13)
> BTW - thanks for testing Gary!
My pleasure.
Updated•11 years ago
|
Target Milestone: --- → Firefox 29
Assignee | ||
Comment 15•11 years ago
|
||
Not seeing any problems with the newest mc nightly I triggered on this push.
Keywords: verifyme
Comment 16•11 years ago
|
||
status-firefox28:
--- → fixed
status-firefox29:
--- → fixed
Comment 17•11 years ago
|
||
Updated•11 years ago
|
status-b2g-v1.3:
--- → fixed
Comment 18•11 years ago
|
||
I couldn't reproduce this issue with the Nightly from 2014-01-31 on Win 8.1 64-bit: after setting Nightly as the default browser and opening a link from Skype, I don't get the error you mention.
Any thoughts/suggestions?
Flags: needinfo?(garyshap)
Comment 20•11 years ago
|
||
I've also tried to reproduce this issue with the Nightly from 2014-02-01, but still without any success.
Gary, could you please check this has been fixed with:
1) latest Beta (28 beta 2): ftp://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/28.0b2-candidates
2) latest Aurora: ftp://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-aurora/
3) latest Nightly: ftp://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central/
Flags: needinfo?(garyshap)
Reporter | ||
Comment 21•11 years ago
|
||
The Nightly from 2014-02-01 shouldn't have the error since it first appeared on inbound on that day and the fix landed on m-c right away. So the original bad patch wasn't on Nightly on that day which is why you can't reproduce it.
I suspect Beta and Aurora either don't have the original bad patch or have the fixed one. In either case you won't get the error.
Flags: needinfo?(garyshap)
Comment 22•11 years ago
|
||
Marking this verified fixed, based on comment 19 and comment 21 and on the fact that I wasn't able to reproduce this issue with neither the Nightly from 2014-01-31 nor the Nightly from 2014-02-01, as mentioned in comment 18 and comment 20.
Thanks Gary for your help.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•