Closed
Bug 386164
Opened 18 years ago
Closed 17 years ago
crashreporter expects "Vendor" to be existing
Categories
(Toolkit :: Crash Reporting, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 396209
People
(Reporter: wolfiR, Unassigned)
Details
crashreporter expects "Vendor" to be existing
Vendor is an optional field in application.ini but if it's not set the crash reporter on Windows saves to the registry key Software\Crash Reporter and therefore treats Vendor to be mandatory.
Reporter | ||
Comment 1•18 years ago
|
||
I wonder if
http://mxr.mozilla.org/seamonkey/source/toolkit/airbag/client/crashreporter_win.cpp#699
was intended to fall back to Software\Mozilla\Crash Reporter but doesn't do it because Vendor could be found in gQueryParameters but is empty?
Reporter | ||
Comment 2•18 years ago
|
||
Aehm, forgot my proposal: Fall back to product/Name/ProductName if Vendor is
empty/not defined.
Comment 3•17 years ago
|
||
Looks like we should probably be testing the string here to see if it's an empty string:
http://mxr.mozilla.org/seamonkey/source/toolkit/xre/nsAppRunner.cpp#2524
But this might be obviated by bent's fix over on bug 396486.
Rolled into the patch on bug 396209.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•