Closed Bug 525725 Opened 10 years ago Closed 10 years ago

xulrunner -help crashes [@ LoadPlatformDirectory]

Categories

(Toolkit :: Startup and Profile System, defect, critical)

x86
Windows XP
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Tracking Status
status1.9.2 --- beta3-fixed

People

(Reporter: neil, Assigned: neil)

References

Details

(Keywords: crash, regression)

Crash Data

Attachments

(1 file)

Access violation - code c0000005
  nsresult rv = aBundleDirectory->Clone(getter_AddRefs(platformDir));
aBundleDirectory is null.

Call stack:
xul!LoadPlatformDirectory
xul!nsXREDirProvider::LoadBundleDirectories
xul!nsXREDirProvider::GetFilesInternal
xul!nsXREDirProvider::GetFiles
xpcom_core!FindProviderFile
xpcom_core!nsSupportsArray::EnumerateBackwards
xpcom_core!nsDirectoryService::Get
xpcom_core!AppendFromDirServiceList
xpcom_core!xptiInterfaceInfoManager::BuildFileSearchPath
xpcom_core!xptiInterfaceInfoManager::GetInterfaceInfoManagerNoAddRef
xpcom_core!NS_InitXPCOM3_P
xul!ScopedXPCOMStartup::Initialize
xul!DumpArbitraryHelp
xul!DumpHelp
xul!XRE_main
seamonkey!NS_internal_main
seamonkey!wmain
seamonkey!__tmainCRTStartup
seamonkey!wmainCRTStartup

So I'm assuming -help doesn't set mXULAppDir, but it should.
Flags: blocking1.9.2?
Attached patch Proposed patchSplinter Review
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #409587 - Flags: review?(benjamin)
Severity: normal → critical
Keywords: crash
Summary: -help crashes → xulrunner -help crashes [@ LoadPlatformDirectory]
Attachment #409587 - Flags: review?(benjamin) → review+
Duplicate of this bug: 525441
Pushed changeset ac5673902abd to mozilla-central.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Not blocking, but we should take the patch.
Flags: blocking1.9.2? → blocking1.9.2-
Pushed changeset eca0dce6296c to releases/mozilla-1.9.2
turns out that lots of people seem to do do this.  its top crash in early 3.6b3 data.  re-asking for blocking so it gets a good flag an back on the radar if a regression happens with the landing.

see Bug 529482 for crash analysis
Flags: blocking1.9.2- → blocking1.9.2?
Flags: blocking1.9.2? → blocking1.9.2+
Duplicate of this bug: 529482
(In reply to comment #7)
> does the patch need to go to other branches?
No, this particular crash should just be a regression from bug 521262.
Crash Signature: [@ LoadPlatformDirectory]
You need to log in before you can comment on or make changes to this bug.