I tried installing the theme I'm working on from the hard disk, using an html file with an InstallTrigger. The url was right, when put in the address bar it opened the file. When on the html page, clicking the InstallTrigger gave me the usual prompt ( "Install theme from location" ), to which I agreed; it starts to fetch the file, and then crashes mozilla. I was able to repeat this issue numerous times.
Reporter: Which build are you using ? Please include in all bug reports the build ID ! BTW: Have you tried file:/// (file:///c:/test.jar) ? Can you attach the trigger-File to this bug (via create attachment)
Severity: normal → critical
Status: UNCONFIRMED → NEW
Component: Installer → Installer: XPInstall Engine
Ever confirmed: true
QA Contact: bugzilla → jimmylee
This is repeatable here with both 0.9.4 (2001091303 - win98) and 0.9.5 (2001101117 - win2000) I don't need to add any more info as Matthias Versen's comment said it all.
David, can you take a look at this?
Assignee: syd → dprice
No problem. The function nsXPInstallManager::OnProgress is passed an nsIRequest* That is then QueryInterface'd to an nsIChannel The QI is failing. The code currently tosses an assert. The quick and dirty fix would be to simply check the result of the QI and fail. nsIRequest is an nsIFileTransport The right thing to do would be to make certain the file transport supports the nsIChannel interface.
Spoke to dveditz about this one. Looks like it is a necko problem Over to darin This is a case where we're getting a null channel that "will never happen"
Assignee: dprice → darin
ok, so the file channel is letting file transport requests slip out... bad file channel! should be an easy fix.
Status: NEW → ASSIGNED
OS: Windows 98 → All
Priority: -- → P2
Target Milestone: --- → mozilla0.9.6
Target Milestone: mozilla0.9.6 → mozilla0.9.8
Changing QA Contact to Ben. Changing Component from XPInstall to Networking. Please update accordingly if I didn't quite get the changes right. Thanks!
Component: Installer: XPInstall Engine → Networking
QA Contact: jimmylee → benc
looks like a dupe of a bug 113949, which i just recently fixed. can someone confirm that this is now fixed in a recent night build? *** This bug has been marked as a duplicate of 113949 ***
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → DUPLICATE
this is fixed for me with an 1h old CVS build. (no crash)
You need to log in before you can comment on or make changes to this bug.